Skip to content

fix(types): update incorrect tracer option names#6526

Merged
watson merged 2 commits intoDataDog:masterfrom
sophiebits:sophiebits/types
Nov 26, 2025
Merged

fix(types): update incorrect tracer option names#6526
watson merged 2 commits intoDataDog:masterfrom
sophiebits:sophiebits/types

Conversation

@sophiebits
Copy link
Copy Markdown
Contributor

These are outdated or typoed. There are other options which are not named in the types which I haven't added in this PR.

@sophiebits sophiebits requested a review from a team as a code owner September 25, 2025 20:17
watson
watson previously approved these changes Sep 26, 2025
Copy link
Copy Markdown
Collaborator

@watson watson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @sophiebits

Thank you for contributing and fixing the types 💯

@codecov
Copy link
Copy Markdown

codecov Bot commented Sep 26, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.08%. Comparing base (9b25bed) to head (c9f448a).
⚠️ Report is 11 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #6526      +/-   ##
==========================================
- Coverage   84.08%   84.08%   -0.01%     
==========================================
  Files         484      484              
  Lines       20277    20277              
==========================================
- Hits        17050    17049       -1     
- Misses       3227     3228       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@sophiebits
Copy link
Copy Markdown
Contributor Author

Thanks for the test fix!

@watson
Copy link
Copy Markdown
Collaborator

watson commented Sep 26, 2025

@sophiebits My pleasure. Since I added a commit, I'm just waiting for a 2nd review from someone else before it can be merged

@BridgeAR
Copy link
Copy Markdown
Member

BridgeAR commented Sep 27, 2025

@sophiebits the PR is highly appreciated! Thank you from me as well :)

Could you verify your signature? We can otherwise not merge the PR as is and would have to rebase on our own and force push.

@BridgeAR BridgeAR enabled auto-merge (squash) September 27, 2025 07:49
@BridgeAR BridgeAR disabled auto-merge September 27, 2025 07:50
sophiebits and others added 2 commits November 26, 2025 12:35
These are outdated or typoed. There are other options which are not named in the types which I haven't added in this PR.

Signed-off-by: Thomas Watson <thomas.watson@datadoghq.com>
Signed-off-by: Thomas Watson <thomas.watson@datadoghq.com>
@watson watson enabled auto-merge (squash) November 26, 2025 11:37
@watson watson merged commit 8796ac8 into DataDog:master Nov 26, 2025
1449 of 1956 checks passed
@watson
Copy link
Copy Markdown
Collaborator

watson commented Nov 26, 2025

Thanks again. I co-signed your commits, so now we could merge it. It will be included in the next release

@sophiebits
Copy link
Copy Markdown
Contributor Author

Thank you!

@sophiebits
Copy link
Copy Markdown
Contributor Author

I missed your comment @BridgeAR, sorry about that.

dd-octo-sts Bot pushed a commit that referenced this pull request Nov 27, 2025
* fix(types): update incorrect tracer option names

These are outdated or typoed. There are other options which are not named in the types which I haven't added in this PR.

Signed-off-by: Thomas Watson <thomas.watson@datadoghq.com>

* Fix type:test

Signed-off-by: Thomas Watson <thomas.watson@datadoghq.com>

---------

Signed-off-by: Thomas Watson <thomas.watson@datadoghq.com>
Co-authored-by: Thomas Watson <thomas.watson@datadoghq.com>
@dd-octo-sts dd-octo-sts Bot mentioned this pull request Nov 27, 2025
BridgeAR pushed a commit that referenced this pull request Dec 17, 2025
* fix(types): update incorrect tracer option names

These are outdated or typoed. There are other options which are not named in the types which I haven't added in this PR.

Signed-off-by: Thomas Watson <thomas.watson@datadoghq.com>

* Fix type:test

Signed-off-by: Thomas Watson <thomas.watson@datadoghq.com>

---------

Signed-off-by: Thomas Watson <thomas.watson@datadoghq.com>
Co-authored-by: Thomas Watson <thomas.watson@datadoghq.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants