fix(libdd-data-pipeline): cargo clippy fix with all lints#2013
fix(libdd-data-pipeline): cargo clippy fix with all lints#2013Eldolfin wants to merge 13 commits into
Conversation
…ebug, items_after_statements, match_wildcard, hidden_lifetime, from_iter, similar_names
…dant_closure, lifetime params, significant_drop_tightening, match_same_arms
…dless_pass_by_value, trivially_copy_pass_by_ref, unused_self, struct_field_names
…ix trivially_copy, needless_pass_by_value, cast_possible_truncation
…eded by workspace lints
📚 Documentation Check Results📦
|
|
Clippy Allow Annotation ReportComparing clippy allow annotations between branches:
Summary by Rule
Annotation Counts by File
Annotation Stats by Crate
About This ReportThis report tracks Clippy allow annotations for specific rules, showing how they've changed in this PR. Decreasing the number of these annotations generally improves code quality. |
🔒 Cargo Deny Results📦
|
…, similar_names rename
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #2013 +/- ##
==========================================
- Coverage 72.68% 72.68% -0.01%
==========================================
Files 452 453 +1
Lines 74930 74932 +2
==========================================
- Hits 54463 54462 -1
- Misses 20467 20470 +3
🚀 New features to boost your workflow:
|
…pecific to 1.84.1, fixed in 1.85.0
Artifact Size Benchmark Reportaarch64-alpine-linux-musl
aarch64-unknown-linux-gnu
libdatadog-x64-windows
libdatadog-x86-windows
x86_64-alpine-linux-musl
x86_64-unknown-linux-gnu
|
these appear only in very specific configurations I can't explain...
What does this PR do?
Applies the restrictive workspace Clippy/Rust lint config and opts in
libdd-data-pipeline, fixing the resulting lints without changing behavior.Motivation
Tighten lint coverage incrementally, crate by crate. APMSP-3056
Additional Notes
How to test the change?
CI passing is sufficient validation.