chore!: remove all unused dependencies - #2319
Conversation
Ran `cargo machete --with-metadata --fix` then fixed clippy lints
🎉 All green!🧪 All tests passed 🎯 Code Coverage (details) 🔗 Commit SHA: c2d069f | Docs | Datadog PR Page | Give us feedback! |
Bits couldn’t fix the CIWarning This comment is related to an earlier commit. 🟢 Investigated · 🔴 Fix failed · ⚪ Validate · ⚪ Ready
View in Datadog | Reviewed commit 42b579e · Any feedback? Reach out in #deveng-pr-agent |
🔒 Cargo Deny Results📦
|
📚 Documentation Check Results📦
|
BenchmarksComparisonBenchmark execution time: 2026-08-06 13:52:45 Comparing candidate commit c2d069f in PR branch Found 15 performance improvements and 14 performance regressions! Performance is the same for 139 metrics, 10 unstable metrics.
|
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
|
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 64e47c7bc3
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
What does this PR do?
Removes all unused dependencies for all the crates.
I ran
cargo machete --with-metadata --fixthen fixed obvious false positive that were pointed out by running clippy.Motivation
Additional Notes
Anything else we should know when reviewing?
How to test the change?
CI should be able to catch if I broke something
TODO