-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
docs: use nightly clippy in vscode #840
Conversation
Before this small fix, I had a bunch of Clippy warnings/errors on VS Code.
Wait, actually, it's not working well |
Codecov Report
@@ Coverage Diff @@
## main #840 +/- ##
==========================================
+ Coverage 73.25% 73.32% +0.07%
==========================================
Files 278 278
Lines 29282 29282
==========================================
+ Hits 21450 21472 +22
+ Misses 7832 7810 -22
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
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'm not a RA user so can't really comment on this
wdyt @rkrasiuk
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 can just add +nightly
"rust-analyzer.checkOnSave.overrideCommand": [ | ||
"cargo", | ||
"+nightly", | ||
"clippy", | ||
"--all", | ||
"--all-features", | ||
"--message-format=json" | ||
], |
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.
"rust-analyzer.checkOnSave.overrideCommand": [ | |
"cargo", | |
"+nightly", | |
"clippy", | |
"--all", | |
"--all-features", | |
"--message-format=json" | |
], | |
"rust-analyzer.checkOnSave.command": "clippy", | |
"rust-analyzer.checkOnSave.features": "all", | |
"rust-analyzer.checkOnSave.extraArgs": ["+nightly"], |
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.
It not working for me:
[ERROR flycheck] Flycheck failed to run the following command: "cargo" "check" "--workspace" "--message-format=json" "--manifest-path" "/workspaces/crypto/learn/ethereum/reth/reth/Cargo.toml" "--all-targets" "+nightly"
[ERROR rust_analyzer::lsp_utils] cargo check failed
Cargo watcher failed, the command produced no valid metadata (exit code: ExitStatus(unix_wait_status(256))):
error: Found argument '+nightly' which wasn't expected, or isn't valid in this context
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.
Sorry, add rust-analyzer.checkOnSave.command
to clippy
as well. Updated my suggestion to include the new things
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.
Hmmm I have a similar error:
[ERROR flycheck] Flycheck failed to run the following command: "cargo" "clippy" "--workspace" "--message-format=json" "--manifest-path" "/workspaces/crypto/learn/ethereum/reth/reth/Cargo.toml" "--all-targets" "--all-features" "+nightly"
[ERROR rust_analyzer::lsp_utils] cargo check failed
Cargo watcher failed, the command produced no valid metadata (exit code: ExitStatus(unix_wait_status(256))):
error: Found argument '+nightly' which wasn't expected, or isn't valid in this context
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.
Alright, strange. This is fine then, thank you for being patient!
Before this small fix, I had a bunch of Clippy warnings/errors on VS Code.