Remove unused Rust dependencies - #61
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- remove cargo-machete-confirmed unused direct dependencies from four Rust crates\n- keep the PR separate from docs PR #60 and base it on the lockfile-normalization branch\n- update only workspace package dependency lists in Cargo.lock; no dependency version churn\n\n## Removed\n- aw-contour-smoke: chrono, url\n- detmir-status: serde\n- hayabusa-tools: urlencoding\n- worktime-api: urlencoding\n\n## Verification\n- manual rg verification for source usage\n- cargo metadata --offline --format-version 1\n- cargo test -p aw-contour-smoke -p detmir-status -p hayabusa-tools -p worktime-api --locked\n- cargo machete\n- git diff --check\n- cargo metadata --locked --format-version 1\n- cargo clippy -p aw-contour-smoke -p detmir-status -p hayabusa-tools -p worktime-api --all-targets --locked -- -D warnings