chore: resolve all clippy lints + remove dead code #14

Closed

gabrielgad wants to merge 1 commit from split/cleanup into split/features

gabrielgad commented

2026-06-30 15:37:06 -04:00

(Migrated from github.com)

cargo clippy --fix for the machine-applicable lints, plus manual fixes (&PathBuf/&Vec → &Path/&[_], sort_by_key, strip_prefix, self-by-value, enumerate, struct-init folds, doc-comment formatting), crate-level allows for the design-inherent lints (too_many_arguments, module_inception, type_complexity), and removal of the never-read PermissionStatus::rules_installed.

Zero rustc + zero clippy warnings; 153 tests pass.

`cargo clippy --fix` for the machine-applicable lints, plus manual fixes (`&PathBuf`/`&Vec` → `&Path`/`&[_]`, `sort_by_key`, `strip_prefix`, self-by-value, `enumerate`, struct-init folds, doc-comment formatting), crate-level allows for the design-inherent lints (`too_many_arguments`, `module_inception`, `type_complexity`), and removal of the never-read `PermissionStatus::rules_installed`. **Zero rustc + zero clippy warnings; 153 tests pass.**

gabrielgad commented

2026-07-03 16:21:25 -04:00

(Migrated from github.com)

Content already fully merged into main (confirmed via git ancestor check) - closing this stale PR record.