This repository was archived by the owner on Nov 24, 2023. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 62
cargo fix --clippy isn't as smart as I want it to be :( #182
Comments
jyn514
added a commit
to jyn514/saltwater
that referenced
this issue
Nov 18, 2019
not bothering to change any more until one of google/rerast#31 or rust-lang/rustfix#182 is fixed
Thanks for the report. I think there are two things here:
|
So if I set |
Yeah, please file an issue on the Clippy repo to improve this suggestion. Clippy suggestions are largely per-case so not all of them exist yet in machine applicable form. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Uh oh!
There was an error while loading. Please reload this page.
This gives a warning instead of replacing it with the suggested code:
It would be really nice to have this be done automatically, since I currently have about 20 of these warnings in my code.
$ cargo +nightly --version cargo 1.40.0-nightly (5da4b4d47 2019-10-28) $ ~/.rustup/toolchains/nightly-x86_64-apple-darwin/bin/rustfmt --version rustfmt 1.4.9-nightly (33e36670 2019-10-07)
The text was updated successfully, but these errors were encountered: