futpib / slopd / 26830324596

86%

Build Type

push

github

Committed by futpib

Commit Message

ci: run clippy with -D warnings and fix existing lints

Adds a clippy job to CI (`cargo clippy --workspace --all-features --all-targets -- -D warnings`) so lint regressions fail the build, and clears the existing debt so it passes.

Mechanical simplifications (unnecessary_map_or, collapsible_if, len_zero, needless_range_loop, double_ended_iterator_last, derivable Default) were applied via `cargo clippy --fix`. The two subscribe-confirmation loops, which never iterated, are rewritten as a single match — fixing the deny-by-default never_loop.

Lock-file opens get an explicit `.truncate(false)` (suspicious_open_options). Genuinely intentional cases keep scoped #[allow]s with a justification: the Option-returning `from_str` helpers (not the std Result-returning trait) and the argument-heavy CLI/daemon dispatch fns.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

Coverage Stats

61 of 83 new or added lines in 6 files covered. (73.49%)

221 existing lines in 4 files now uncovered.

3686 of 4359 relevant lines covered (84.56%)

985514.75 hits per line