Skip to content

chore: Remove more old transaction stuff - #7492

Merged
sentrivana merged 37 commits into
major/3.0from
ivana/major/more-old-tracing
Sep 15, 2026
Merged

sentrivana merged 37 commits into
major/3.0from
ivana/major/more-old-tracing

Conversation

@sentrivana

@sentrivana sentrivana commented Sep 14, 2026

Copy link
Copy Markdown
Contributor
  • remove scope.transaction (set_transaction_name remains for now; changed all direct uses of scope.transaction = to use set_transaction_name instead)
  • remove more interop utils (_serialized_v1_attribute_to_serialized_v2_attribute etc.)
  • remove event processor-based transaction name setting (superseded by set_transaction_name directly in patches)
  • ...

@github-actions

github-actions Bot commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

Codecov Results 📊

64517 passed | ❌ 1 failed | ⏭️ 3049 skipped | Total: 67567 | Pass Rate: 95.49% | Execution Time: 183m 45s

📊 Comparison with Base Branch

Metric Change
Total Tests 📉 -67885
Passed Tests 📉 -63804
Failed Tests 📈 +1
Skipped Tests 📉 -4082

❌ Patch coverage is 58.33%. Project has 2101 uncovered lines.
❌ Project coverage is 90.11%. Comparing base (d68570b) to head (724608f).

Files with missing lines (2)
File Patch % Lines
sentry_sdk/integrations/django/init.py 0.00% ⚠️ 4 Missing
sentry_sdk/api.py 0.00% ⚠️ 1 Missing
Coverage diff
@@            Coverage Diff             @@
##        master       #PR       +/-##
==========================================
- Coverage    90.18%    90.11%    -0.07%
==========================================
  Files          193       180       -13
  Lines        25768     21237     -4531
  Branches      9532      7666     -1866
==========================================
+ Hits         23236     19136     -4100
- Misses        2532      2101      -431
- Partials      1432      1211      -221

Generated by Codecov Action

Base automatically changed from ivana/major/rip-out-old-tracing to major/3.0 September 14, 2026 09:12
@sentrivana
sentrivana marked this pull request as ready for review September 14, 2026 12:41
@sentrivana
sentrivana requested a review from a team as a code owner September 14, 2026 12:41

@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 492bde5. Configure here.

Comment thread sentry_sdk/integrations/bottle.py
@sentrivana
sentrivana marked this pull request as draft September 14, 2026 12:58
@sentrivana
sentrivana marked this pull request as ready for review September 15, 2026 09:17
@sentrivana
sentrivana merged commit bd4d825 into major/3.0 Sep 15, 2026
278 of 287 checks passed
@sentrivana
sentrivana deleted the ivana/major/more-old-tracing branch September 15, 2026 10:53
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