-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Clippy fixes #10113
Clippy fixes #10113
Conversation
Which will be optimized away by the compiler
using panic! instead with a string message
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #10113 +/- ##
==========================================
- Coverage 82.19% 82.11% -0.09%
==========================================
Files 975 974 -1
Lines 271940 271869 -71
==========================================
- Hits 223523 223242 -281
- Misses 48417 48627 +210
Flags with carried forward coverage won't be shown. Click here to find out more. |
I don't think CI does |
I do |
ci also does --all-features |
Still get it with |
cargo clippy --version |
I think those lints are reported in CI, they just don't make edits so we don't fail out on them. Is that the difference? |
So clippy does not return 0 (maybe because we have 2 crates (derive and suricata) in our worskspace ? ) |
Still probably good to make it clean for when running |
Information: QA ran without warnings. Pipeline 17340 |
Continued in #10124 |
Link to redmine ticket:
None
Describe changes:
@jasonish why do I get these and CI is not red ?
Because I am using a newer clippy ?
I also have others like zero_prefixed_literal