Skip to content

chore: Drop tags auto-set by the SDK - #7261

Open
sentrivana wants to merge 4 commits into
major/3.0from
ivana/major/remove-tags
Open

chore: Drop tags auto-set by the SDK#7261
sentrivana wants to merge 4 commits into
major/3.0from
ivana/major/remove-tags

Conversation

@sentrivana

@sentrivana sentrivana commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Description

Many of the set_tag calls were only present in non-span-streaming branches, which will be completely removed anyway, and we've already migrated those tags, if applicable, to span attributes in the scope of span first.

Issues

Closes https://linear.app/getsentry/issue/PY-1940/remove-tags-auto-set-by-the-sdk

@linear-code

linear-code Bot commented Aug 26, 2026

Copy link
Copy Markdown

PY-1940

@github-actions

github-actions Bot commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Codecov Results 📊

110594 passed | ⏭️ 5428 skipped | Total: 116022 | Pass Rate: 95.32% | Execution Time: 384m 4s

📊 Comparison with Base Branch

Metric Change
Total Tests 📉 -8574
Passed Tests 📉 -7270
Failed Tests
Skipped Tests 📉 -1304

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 2257 uncovered lines.
✅ Project coverage is 90.55%. Comparing base (base) to head (head).

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    90.25%    90.55%     +0.3%
==========================================
  Files          193       185        -8
  Lines        25441     23878     -1563
  Branches      9352      8794      -558
==========================================
+ Hits         22960     21621     -1339
- Misses        2481      2257      -224
- Partials      1436      1377       -59

Generated by Codecov Action

@sentrivana
sentrivana marked this pull request as ready for review August 26, 2026 14:19
@sentrivana
sentrivana requested a review from a team as a code owner August 26, 2026 14:19
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.

2 participants