Skip to content

Support multiple tracing header types in DdUrlTransfer#181

Merged
ambushwork merged 3 commits into
developfrom
yl/support-multi-header-types
Jun 16, 2026
Merged

Support multiple tracing header types in DdUrlTransfer#181
ambushwork merged 3 commits into
developfrom
yl/support-multi-header-types

Conversation

@ambushwork

@ambushwork ambushwork commented Jun 8, 2026

Copy link
Copy Markdown
Member

What does this PR do?

See #171

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Make sure you discussed the feature or bugfix with the maintaining team in an Issue
  • Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

@ambushwork
ambushwork force-pushed the yl/support-multi-header-types branch from 4d96aca to e377eec Compare June 12, 2026 15:22
@datadog-datadog-prod-us1

This comment has been minimized.

@ambushwork
ambushwork force-pushed the yl/support-multi-header-types branch from e377eec to ce2a991 Compare June 12, 2026 15:35
@ambushwork

Copy link
Copy Markdown
Member Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: ce2a991051

ℹ️ 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".

Comment thread README.md
@ambushwork
ambushwork marked this pull request as ready for review June 15, 2026 10:02
@ambushwork
ambushwork requested review from a team as code owners June 15, 2026 10:02
@domalessi domalessi self-assigned this Jun 15, 2026
domalessi
domalessi previously approved these changes Jun 15, 2026
kikoveiga
kikoveiga previously approved these changes Jun 16, 2026
Comment thread library/source/wrapper/DdUrlTransfer.bs Outdated
Comment thread library/source/wrapper/DdUrlTransfer.bs
@ambushwork
ambushwork dismissed stale reviews from kikoveiga and domalessi via df9b01d June 16, 2026 12:52
@ambushwork
ambushwork force-pushed the yl/support-multi-header-types branch from ce2a991 to df9b01d Compare June 16, 2026 12:52
@ambushwork
ambushwork merged commit 1a6c0a8 into develop Jun 16, 2026
8 checks passed
@ambushwork
ambushwork deleted the yl/support-multi-header-types branch June 16, 2026 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants