-
Notifications
You must be signed in to change notification settings - Fork 266
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
build(rust): Trigger warnings for unused crate dependencies #4402
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Ignored Deployment
|
Terraform Cloud Plan Output
|
Performance Test ResultsTCP
UDP
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've had success with https://github.com/est31/cargo-udeps in the past.
@@ -34,7 +34,9 @@ jobs: | |||
cargo doc --all-features --no-deps --document-private-items ${{ matrix.packages }} | |||
env: | |||
RUSTDOCFLAGS: "-D warnings" | |||
- run: | | |||
- env: | |||
RUSTFLAGS: "-Wunused-crate-dependencies" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You should be able to set this lint in Cargo.toml
under the recently introduced lints
table.
Superseded by #4475 |
After all of the time spent hunting down all of our
fstat
calls for #4377, it became clear that our dependency tree is somewhat convoluted and includes a few unused crates and features.This is a first step to try to clean up some of those.
Refs #4403