Skip to content

chore(rq,arq,pyreqwest): Run blocks gated by has_span_streaming_enabled() unconditionally - #7443

Merged
alexander-alderman-webb merged 4 commits into
major/3.0from
webb/remove-has-span-streaming-enabled-branches
Sep 9, 2026
Merged

chore(rq,arq,pyreqwest): Run blocks gated by has_span_streaming_enabled() unconditionally#7443
alexander-alderman-webb merged 4 commits into
major/3.0from
webb/remove-has-span-streaming-enabled-branches

Conversation

@alexander-alderman-webb

Copy link
Copy Markdown
Contributor

Description

These were an oversight when removing transaction-based tracing API.

Issues

Reminders

@alexander-alderman-webb
alexander-alderman-webb requested a review from a team as a code owner September 9, 2026 11:14

@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 9b80f2d. Configure here.

Comment thread sentry_sdk/integrations/rq.py
@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Codecov Results 📊

71604 passed | ⏭️ 3374 skipped | Total: 74978 | Pass Rate: 95.5% | Execution Time: 222m 19s

📊 Comparison with Base Branch

Metric Change
Total Tests 📉 -64058
Passed Tests 📉 -60301
Failed Tests
Skipped Tests 📉 -3757

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 2250 uncovered lines.
❌ Project coverage is 90.15%. Comparing base (490cff3) to head (b47b8e6).

Files with missing lines (1)
File Patch % Lines
sentry_sdk/integrations/arq.py 100.00% ⚠️ 1 partials
Coverage diff
@@            Coverage Diff             @@
##        master       #PR       +/-##
==========================================
- Coverage    90.20%    90.15%    -0.05%
==========================================
  Files          193       185        -8
  Lines        25788     22852     -2936
  Branches      9530      8338     -1192
==========================================
+ Hits         23261     20602     -2659
- Misses        2527      2250      -277
- Partials      1432      1343       -89

Generated by Codecov Action

@alexander-alderman-webb
alexander-alderman-webb merged commit d0cd363 into major/3.0 Sep 9, 2026
149 checks passed
@alexander-alderman-webb
alexander-alderman-webb deleted the webb/remove-has-span-streaming-enabled-branches branch September 9, 2026 12:03
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