-
Notifications
You must be signed in to change notification settings - Fork 236
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
Enforce clippy #2614
Enforce clippy #2614
Conversation
Codecov ReportBase: 67.66% // Head: 67.97% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #2614 +/- ##
==========================================
+ Coverage 67.66% 67.97% +0.30%
==========================================
Files 200 200
Lines 29279 29278 -1
Branches 5755 5759 +4
==========================================
+ Hits 19813 19903 +90
+ Misses 4870 4776 -94
- Partials 4596 4599 +3
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. ☔ View full report at Codecov. |
Just wanted to comment while I was thinking about it: I think we should either make this test not-required, or set a fixed rust version in the CI. We don't want our CI to start failing because a new clippy version was released which adds new warnings, similar to how we don't fail on code warnings, since sometimes newer rust versions add new warnings and we don't want our tests to fail because of them. |
Ok, I think I've got this sorted out now. I ultimately added a rust-toolchain.toml file to set the default toolchain in general, since it is an issue we've run into with other lints etc. It can be overridden locally various ways if desired, but it probably makes sense to mostly just have everything use that one version. LMK whether you'd like me to clean up the history a bit before you review/continue-to-review |
Also always set rust profile to minimal.
Just a note: Each time we change the rust version in |
Fixes remaining clippy warnings and adds a clippy job to our lint github workflow. Also pins the default Rust version with a rust-toolchain.yaml file.