Skip to content

Fix all clippy warnings and rustfmt across workspace#3

Merged
colinrozzi merged 1 commit intomainfrom
clippy-fmt-fixes
Apr 27, 2026
Merged

Fix all clippy warnings and rustfmt across workspace#3
colinrozzi merged 1 commit intomainfrom
clippy-fmt-fixes

Conversation

@colinrozzi
Copy link
Copy Markdown
Owner

Summary

  • Resolve all 26 clippy lint errors (with -D warnings) across the workspace
  • Apply cargo fmt to fix formatting inconsistencies (byte array layouts, chain expressions, etc.)
  • All three CI checks now pass: cargo check, cargo clippy, cargo test

Key fixes

  • #[derive(Default)] for Encoder instead of manual impl
  • Collapse nested matches and else { if } blocks
  • Replace redundant closures with function references
  • Remove dead decode_type function
  • Fix clone() on Copy types
  • push_str("\n")push('\n')
  • Use char array patterns instead of chained || comparisons
  • #[allow(clippy::too_many_arguments)] for cross_package_call

Test plan

  • cargo clippy --workspace -- -D warnings passes
  • cargo fmt --all -- --check passes
  • cargo test --workspace — all 177 tests passing

🤖 Generated with Claude Code

@colinrozzi colinrozzi enabled auto-merge April 27, 2026 06:13
Resolve all clippy lints and apply rustfmt so CI checks pass cleanly.
Switch CI from dtolnay/rust-toolchain to nix develop so the runner
uses the exact same toolchain as the flake, avoiding version skew.
@colinrozzi colinrozzi merged commit 974a078 into main Apr 27, 2026
2 checks passed
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