Skip to content

feat(span-streaming): Add more attrs to segment spans#6432

Merged
sentrivana merged 5 commits into
masterfrom
ivana/add-more-attrs-to-segment-spans
May 28, 2026
Merged

feat(span-streaming): Add more attrs to segment spans#6432
sentrivana merged 5 commits into
masterfrom
ivana/add-more-attrs-to-segment-spans

Conversation

@sentrivana

@sentrivana sentrivana commented May 28, 2026

Copy link
Copy Markdown
Contributor

Add missing attrs to segment spans: sentry.release, sentry.sdk.integrations, sentry.platform, sentry.dist, sentry.environment

@sentrivana sentrivana marked this pull request as ready for review May 28, 2026 08:33
@sentrivana sentrivana requested a review from a team as a code owner May 28, 2026 08:33
@github-actions

github-actions Bot commented May 28, 2026

Copy link
Copy Markdown
Contributor

Codecov Results 📊

446 passed | Total: 446 | Pass Rate: 100% | Execution Time: 1m 13s

All tests are passing successfully.

✅ Patch coverage is 92.59%. Project has 14909 uncovered lines.

Files with missing lines (2)
File Patch % Lines
sentry_sdk/traces.py 83.33% ⚠️ 1 Missing and 1 partials
sentry_sdk/tracing_utils.py 0.00% ⚠️ 1 Missing

Generated by Codecov Action

Comment thread sentry_sdk/traces.py Outdated
Comment thread sentry_sdk/traces.py

@cursor cursor 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.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 43b8e40. Configure here.

Comment thread sentry_sdk/traces.py Outdated
@sentrivana sentrivana enabled auto-merge (squash) May 28, 2026 08:44
@sentrivana sentrivana merged commit 54f7686 into master May 28, 2026
157 checks passed
@sentrivana sentrivana deleted the ivana/add-more-attrs-to-segment-spans branch May 28, 2026 09:01
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.

2 participants