search

bnjbvr / cargo-machete

Remove unused Rust dependencies with this one weird trick!
MIT License
784 stars 28 forks source link

Rewrite cargo-metadata support #122

Closed g2p closed 2 months ago

g2p commented 5 months ago

This makes cargo machete --with-metadata --fix work; it previously broke with kebab-case package names and packages like xml-rs.

It also removes a hack that attempted to parse opaque source ids (last updated in #106).

The solution involves looking at .packages and not just .resolved.

DenisGorbachev commented 4 months ago

Experienced this bug too, hope that this pull request fixes is

bnjbvr commented 2 months ago

I'll address the comments to help with merging this. Thanks again!

bnjbvr commented 2 months ago

Hmm it seems I can't push to your fork, so I've opened #132. Thanks again!