Move TracerSettings helper methods to TestHelpers project#8375
Move TracerSettings helper methods to TestHelpers project#8375andrewlock merged 4 commits intomasterfrom
TracerSettings helper methods to TestHelpers project#8375Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 2d7decb08e
ℹ️ 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".
Co-authored-by: Raphaël Vandon <raphael.vandon@datadog.com>
60eceaf to
c43d256
Compare
BenchmarksBenchmark execution time: 2026-03-27 15:23:15 Comparing candidate commit c43d256 in PR branch Found 5 performance improvements and 9 performance regressions! Performance is the same for 259 metrics, 15 unstable metrics.
|
Summary of changes
Move the
TracerSettings.Create()methods which are only there for testing convenience into theTestHelpersprojectReason for change
They don't need to be in the main binary
Implementation details
Test coverage
Covered by existing