Skip to content

ref: Remove all uses of sentry.transaction#6237

Merged
loewenheim merged 4 commits into
masterfrom
sebastian/remove-sentry-transaction
Jul 16, 2026
Merged

ref: Remove all uses of sentry.transaction#6237
loewenheim merged 4 commits into
masterfrom
sebastian/remove-sentry-transaction

Conversation

@loewenheim

Copy link
Copy Markdown
Contributor

In the product sentry.transaction should be transparently treated as a synonym of sentry.segment.name (via sentry-conventions and coalescing). This therefore removes all code writing this tag/attribute.

In the product `sentry.transaction` should be transparently treated as
a synonym of `sentry.segment.name` (via `sentry-conventions` and
coalescing). This therefore removes all code writing this tag/attribute.
@loewenheim
loewenheim requested a review from a team as a code owner July 15, 2026 17:06
@loewenheim loewenheim self-assigned this Jul 15, 2026
@loewenheim
loewenheim requested a review from mjq July 15, 2026 17:11

@jjbayer jjbayer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM but I'll let @mjq have the last word on whether the product can support this change.

@loewenheim
loewenheim enabled auto-merge July 16, 2026 11:40
@loewenheim
loewenheim added this pull request to the merge queue Jul 16, 2026
Merged via the queue into master with commit 963e402 Jul 16, 2026
35 checks passed
@loewenheim
loewenheim deleted the sebastian/remove-sentry-transaction branch July 16, 2026 12:29
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