fix(libdd-trace-obfuscation): cargo clippy fix with all lints#1947
fix(libdd-trace-obfuscation): cargo clippy fix with all lints#1947Eldolfin wants to merge 19 commits into
Conversation
📚 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📦
|
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #1947 +/- ##
==========================================
+ Coverage 72.63% 72.64% +0.01%
==========================================
Files 448 448
Lines 73582 73585 +3
==========================================
+ Hits 53444 53458 +14
+ Misses 20138 20127 -11
🚀 New features to boost your workflow:
|
🎉 All green!❄️ No new flaky tests detected 🎯 Code Coverage (details) 🔗 Commit SHA: c352714 | Docs | Datadog PR Page | Give us feedback! |
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
|
ca1e0f5 to
bb56e9b
Compare
| [package] | ||
| name = "libdd-trace-obfuscation" | ||
| version = "2.0.0" | ||
| version = "2.1.0" |
There was a problem hiding this comment.
Where did this come from? The current released version is 2.0.0. @iunanua any idea?
There was a problem hiding this comment.
Looking closer, adding the #[must_use] to fix the clippy::must_use_candidate attribute is considered a breaking change from the semver checks perspective.
What does this PR do?
Applies a restrictive workspace Clippy/Rust lint config and opts in
libdd-trace-obfuscation, fixing the resulting crate-local lints without changing the behavior.Motivation
Tighten lint coverage incrementally by starting with a single crate.
Additional Notes
How to test the change?
CI passing is sufficient validation.