Skip to content

refactor: apply cargo qual code quality rules#353

Merged
RAprogramm merged 6 commits intomainfrom
quality-fixes
Dec 30, 2025
Merged

refactor: apply cargo qual code quality rules#353
RAprogramm merged 6 commits intomainfrom
quality-fixes

Conversation

@RAprogramm
Copy link
Owner

Summary

  • Remove 1400+ empty lines from function bodies per cargo qual rules
  • Move inline comments to doc blocks where appropriate
  • Simplify path imports using local use statements
  • Clean up test files by removing redundant step comments

Stats

  • 1620 → 1 issues (remaining 1 is intentional example comment)
  • 99 files changed
  • 1661 lines deleted, 119 added

Test plan

  • All tests pass (cargo test --all-features)
  • Clippy clean (cargo clippy --all-targets --all-features -- -D warnings)
  • Formatting verified (cargo +nightly fmt)
  • REUSE compliant

- Remove 1400+ empty lines from function bodies
- Move inline comments to doc blocks where appropriate
- Simplify path imports using local use statements
- Clean up test files by removing redundant step comments

All tests pass, clippy clean.
@codecov
Copy link

codecov bot commented Dec 30, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

dependabot bot and others added 5 commits December 30, 2025 09:49
* chore: update CHANGELOG.md [skip ci]

* chore(deps): bump the minor-and-patch group with 5 updates

Bumps the minor-and-patch group with 5 updates:

| Package | From | To |
| --- | --- | --- |
| [serde_json](https://github.com/serde-rs/json) | `1.0.147` | `1.0.148` |
| [itoa](https://github.com/dtolnay/itoa) | `1.0.16` | `1.0.17` |
| [ryu](https://github.com/dtolnay/ryu) | `1.0.21` | `1.0.22` |
| [proc-macro2](https://github.com/dtolnay/proc-macro2) | `1.0.103` | `1.0.104` |
| [axum-test](https://github.com/JosephLenton/axum-test) | `18.4.1` | `18.5.0` |


Updates `serde_json` from 1.0.147 to 1.0.148
- [Release notes](https://github.com/serde-rs/json/releases)
- [Commits](serde-rs/json@v1.0.147...v1.0.148)

Updates `itoa` from 1.0.16 to 1.0.17
- [Release notes](https://github.com/dtolnay/itoa/releases)
- [Commits](dtolnay/itoa@1.0.16...1.0.17)

Updates `ryu` from 1.0.21 to 1.0.22
- [Release notes](https://github.com/dtolnay/ryu/releases)
- [Commits](dtolnay/ryu@1.0.21...1.0.22)

Updates `proc-macro2` from 1.0.103 to 1.0.104
- [Release notes](https://github.com/dtolnay/proc-macro2/releases)
- [Commits](dtolnay/proc-macro2@1.0.103...1.0.104)

Updates `axum-test` from 18.4.1 to 18.5.0
- [Commits](https://github.com/JosephLenton/axum-test/commits)

---
updated-dependencies:
- dependency-name: serde_json
  dependency-version: 1.0.148
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: minor-and-patch
- dependency-name: itoa
  dependency-version: 1.0.17
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: minor-and-patch
- dependency-name: ryu
  dependency-version: 1.0.22
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: minor-and-patch
- dependency-name: proc-macro2
  dependency-version: 1.0.104
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: minor-and-patch
- dependency-name: axum-test
  dependency-version: 18.5.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: minor-and-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: RA <70325462+RAprogramm@users.noreply.github.com>
@RAprogramm RAprogramm merged commit 1e562bd into main Dec 30, 2025
28 checks passed
@RAprogramm RAprogramm deleted the quality-fixes branch December 30, 2025 03:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant