-
Notifications
You must be signed in to change notification settings - Fork 746
CI: deny all Clippy warnings including clippy::incompatible_msrv #2310
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
Conversation
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.
If clippy/CI are happy with this change, so am I. Thanks!
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2310 +/- ##
=======================================
Coverage 94.82% 94.82%
=======================================
Files 104 104
Lines 24100 24100
=======================================
Hits 22853 22853
Misses 1247 1247 ☔ View full report in Codecov by Sentry. |
Benchmark resultsInstruction countsSignificant differencesClick to expand
Other differencesClick to expand
Wall-timeSignificant differencesThere are no significant wall-time differences Other differencesClick to expand
Additional informationCheckout details:
|
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.
Neat, I guess the reason for this has gone away -- though I don't think clippy has addressed the underlying false positive. But we can put this back if it is ever needed.
It looks to me like there is no need to ignore
clippy::incompatible_msrv
in Clippy CI task to succeed with--deny warnings
.Context comes from this comment: #2285 (comment)
For PR #2285 I would like to update CI to allow a single warning in most crates & deny all other warnings. I have already tried this with XXX TODO comments; PR #2285 still has green status with these updates.
Considering that ignoring
clippy::incompatible_msrv
is not what we want for PR #2285, and that CI can succeed without ignoring this warning, I think we should remove this allow option as proposed here. I would also like to keep this update separate from PR #2285.