Add Rust dependency hygiene CI - #62
Conversation
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
$## Summary\n- add dedicated Dependency hygiene GitHub Actions workflow\n- enforce cargo machete --with-metadata as a blocking unused-dependency check\n- run cargo metadata, cargo tree --duplicates, cargo audit and cargo deny in dependency hygiene CI\n- add separate nightly cargo udeps advisory job for workflow_dispatch/schedule with continue-on-error\n- require adjacent TOML comments for cargo-machete ignored entries so exceptions are explicit\n\n## Cleanup needed for the new gate\n- remove unused tempfile dev-dependencies found only by cargo machete --with-metadata from aw-contour-smoke, aw-health-check, check-aw-data, check-aw-full, diag-and-manual-restart and rdp-worktime-report\n- update Cargo.lock package dependency lists only; no dependency version churn\n\n## Verification\n- git diff --check\n- YAML parsed with Ruby Psych\n- cargo metadata --locked --format-version 1\n- cargo machete --with-metadata\n- cargo tree --duplicates --locked\n- cargo test -p aw-contour-smoke -p aw-health-check -p check-aw-data -p check-aw-full -p diag-and-manual-restart -p rdp-worktime-report --locked\n- cargo clippy -p aw-contour-smoke -p aw-health-check -p check-aw-data -p check-aw-full -p diag-and-manual-restart -p rdp-worktime-report --all-targets --locked -- -D warnings\n- cargo audit --deny warnings\n- cargo deny --manifest-path adk-rust/Cargo.toml check --config deny.toml --hide-inclusion-graph --show-stats