Skip to content

ref: Remove flag storage from StreamedSpan (1)#5588

Merged
sentrivana merged 1 commit intomasterfrom
ivana/span-first-1-remove-flags
Mar 5, 2026
Merged

ref: Remove flag storage from StreamedSpan (1)#5588
sentrivana merged 1 commit intomasterfrom
ivana/span-first-1-remove-flags

Conversation

@sentrivana
Copy link
Contributor

@sentrivana sentrivana commented Mar 5, 2026

Description

Remove flags support from StreamedSpan for now. We will re-evaluate if we need this later.

Issues

Reminders

@github-actions
Copy link
Contributor

github-actions bot commented Mar 5, 2026

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


This PR will not appear in the changelog.


🤖 This preview updates automatically when you update the PR.

@sentrivana sentrivana changed the title ref: Remove flag storage from StreamedSpan ref: Remove flag storage from StreamedSpan (1) Mar 5, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Mar 5, 2026

Codecov Results 📊

1707 passed | ⏭️ 184 skipped | Total: 1891 | Pass Rate: 90.27% | Execution Time: 2m 17s

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 11869 uncovered lines.

Files with missing lines (1)
File Patch % Lines
traces.py 47.50% ⚠️ 42 Missing

Generated by Codecov Action

@sentrivana sentrivana marked this pull request as ready for review March 5, 2026 11:22
@sentrivana sentrivana requested a review from a team as a code owner March 5, 2026 11:22
@sentrivana sentrivana merged commit 4d9394d into master Mar 5, 2026
161 of 163 checks passed
@sentrivana sentrivana deleted the ivana/span-first-1-remove-flags branch March 5, 2026 11:41
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