FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

feat(streaming): Promote `before_send_span` to top-level option by sentrivana · Pull Request #6885 · getsentry/sentry-python · GitHub

feat(streaming): Promote before_send_span to top-level option - #6885

Merged
sentrivana merged 3 commits into
masterfrom
ivana/before-send-span-top-level
Jul 27, 2026
Merged

feat(streaming): Promote before_send_span to top-level option#6885
sentrivana merged 3 commits into
masterfrom
ivana/before-send-span-top-level

Conversation

sentrivana commented Jul 24, 2026
edited
Loading

Copy link
Copy Markdown
Contributor

Bump before_send_span to a top-level option alongside its other span streaming friends.

Add `before_send_span` as a top-level `sentry_sdk.init()` parameter
alongside `before_send_log` and `before_send_metric`, while keeping
backward compatibility with the `_experiments` dict. Warn when span
streaming is not enabled.
sentrivana marked this pull request as ready for review July 24, 2026 13:00
sentrivana requested a review from a team as a code owner July 24, 2026 13:00

Copy link
Copy Markdown
Contributor

Codecov Results 📊

96545 passed | ⏭️ 6340 skipped | Total: 102885 | Pass Rate: 93.84% | Execution Time: 336m 55s

📊 Comparison with Base Branch

Metric Change
Total Tests 📈 +110
Passed Tests 📈 +110
Failed Tests 📉 -1
Skipped Tests 📈 +1

All tests are passing successfully.

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

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    89.68%    89.68%        —%
==========================================
  Files          193       193         —
  Lines        24330     24332        +2
  Branches      8578      8580        +2
==========================================
+ Hits         21820     21822        +2
- Misses        2510      2510         —
- Partials      1397      1397         —

Generated by Codecov Action

sentrivana merged commit 5204ffa into master Jul 27, 2026
145 checks passed
sentrivana deleted the ivana/before-send-span-top-level branch July 27, 2026 07:16
mgaligniana pushed a commit to mgaligniana/sentry-python that referenced this pull request Aug 9, 2026
…entry#6885)

Bump `before_send_span` to a top-level option alongside its other span
streaming friends.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL