chore!: remove unused dependencies (v2) - #2511
Conversation
🔒 Cargo Deny Results📦
|
📚 Documentation Check Results📦
|
|
✅ All CI checks and tests passed. 🎉 All green!🧪 All tests passed 🎯 Code Coverage (details) 🔗 Commit SHA: f0bca2c | Docs | View more details | 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
|
BenchmarksComparisonBenchmark execution time: 2026-09-11 15:29:54 Comparing candidate commit f0bca2c in PR branch Found 6 performance improvements and 13 performance regressions! Performance is the same for 157 metrics, 0 unstable metrics.
|
uuid already enables getrandom/js for telemetry and data pipeline. Forward Ring's WASM support through libdd-tuf in remote config.
rustls-platform-verifier uses rustls-native-certs transitively. Keep the\ndirect dependency below 0.8.3 to avoid openssl-probe 0.2 certificate\ndirectory scanning.
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
More details
The removed dependencies have no remaining direct, build, target, or feature use. The lockfile and license changes agree with the manifest changes.
🤖 Datadog Autotest · Commit f0bca2c · What is Autotest? · @DataDog review to ask questions · Any feedback? Reach out in #autotest
What does this PR do?
Removes unused dependencies across the workspace.
This is a v2 of #2319. I ran cargo machete with metadata, then audited every finding to preserve false positives required by build scripts, targets, features, and workspace resolution.
Motivation
Additional Notes
The cargo-machete allow list will be added in a follow-up.
libdd-common/Cargo.toml, I'm not sure the removal oftokio-rustlsandrustls-native-certsis good. However it didn't seem to have broken testsHow to test the change?
CI should be able to catch if I broke something.
APMSP-3056
[Generated using codex]