Skip to content

mctp-estack, standalone: remove unused dependencies.#24

Merged
mkj merged 1 commit intoCodeConstruct:mainfrom
nnethercote:deps
Aug 8, 2025
Merged

mctp-estack, standalone: remove unused dependencies.#24
mkj merged 1 commit intoCodeConstruct:mainfrom
nnethercote:deps

Conversation

@nnethercote
Copy link
Copy Markdown
Contributor

Found with cargo-shear and cargo-udeps.

Found with `cargo-shear` and `cargo-udeps`.

Signed-off-by: Nicholas Nethercote <n.nethercote@gmail.com>
@mkj mkj merged commit 014af7f into CodeConstruct:main Aug 8, 2025
4 checks passed
@mkj
Copy link
Copy Markdown
Member

mkj commented Aug 8, 2025

Thanks! I'll try adding cargo-udeps to CI.

@nnethercote
Copy link
Copy Markdown
Contributor Author

There is also cargo-machete. Unfortunately all three tools have lots of false positives and negatives. Using them together manually works ok, but automating might be hard.

@nnethercote nnethercote deleted the deps branch August 8, 2025 07:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants