Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Inject the _dd.dbm_trace_injected tag only when full propagation is enabled #2346

Merged
merged 1 commit into from
Nov 10, 2023
Merged

Inject the _dd.dbm_trace_injected tag only when full propagation is enabled #2346

merged 1 commit into from
Nov 10, 2023

Conversation

bwoebi
Copy link
Collaborator

@bwoebi bwoebi commented Nov 3, 2023

Description

Minor logic fix to conform to our internal specification.

Readiness checklist

  • (only for Members) Changelog has been added to the release document.
  • Tests added for this feature/bug.

Reviewer checklist

  • Appropriate labels assigned.
  • Milestone is set.
  • Changelog has been added to the release document. For community contributors the reviewer is in charge of this task.

@bwoebi bwoebi added this to the 0.94.0 milestone Nov 3, 2023
@bwoebi bwoebi requested a review from a team as a code owner November 3, 2023 14:26
pierotibou
pierotibou approved these changes Nov 3, 2023
View reviewed changes
Copy link
Collaborator

@pierotibou pierotibou left a comment

Choose a reason for hiding this comment

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

:cough: No tests to change? :cough:

@bwoebi bwoebi merged commit 305db66 into master Nov 10, 2023
476 of 511 checks passed
@bwoebi bwoebi deleted the bob/dbm-full-tag branch November 10, 2023 14:33
@bwoebi bwoebi mentioned this pull request Nov 20, 2023
Merged
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pierotibou pierotibou pierotibou approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
0.94.0
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@bwoebi @pierotibou