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
fix: fix feature = "cargo-clippy" deprecation #10679
Conversation
Thanks for the PR, it looks good. I'm assuming that you do not plan on adding more commits to this PR and I will click the "merge when ready" button. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #10679 +/- ##
==========================================
- Coverage 72.28% 72.25% -0.03%
==========================================
Files 735 735
Lines 150535 150535
Branches 150535 150535
==========================================
- Hits 108809 108772 -37
- Misses 36790 36824 +34
- Partials 4936 4939 +3
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@@ -8,9 +8,9 @@ | |||
#![deny(missing_docs, trivial_numeric_casts, unused_extern_crates)] | |||
#![warn(unused_import_braces)] | |||
#![deny(unstable_features)] | |||
#![cfg_attr(feature = "cargo-clippy", allow(clippy::new_without_default))] | |||
#![cfg_attr(clippy, allow(clippy::new_without_default))] |
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.
These no longer need to be cfg_attr
d at all to the best of my knowledge. Fine to leave as-is too though.
Looks like the CI is failing due to formatting. |
Head branch was pushed to by a user without write access
Should be ok now |
https://blog.rust-lang.org/2024/02/28/Clippy-deprecating-feature-cargo-clippy.html