Skip to content

ref(tracing): No-op and emit warning in start_transaction with the streaming trace lifecycle#6845

Open
alexander-alderman-webb wants to merge 13 commits into
masterfrom
webb/no-op-start-transaction
Open

ref(tracing): No-op and emit warning in start_transaction with the streaming trace lifecycle#6845
alexander-alderman-webb wants to merge 13 commits into
masterfrom
webb/no-op-start-transaction

Conversation

@alexander-alderman-webb

@alexander-alderman-webb alexander-alderman-webb commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Description

Mirror the deprecation warning in Scope.start_span(). The API is not available when the streaming trace lifecycle is used.

Adapt tests that wrongly assert on transactions when the lifecycle is enabled.

Issues

Reminders

@alexander-alderman-webb alexander-alderman-webb changed the title ref(tracing): No-op and emit warning in start_transaction with the streaming trace lifecycle ref(tracing): No-op and emit warning in start_transaction with the streaming trace lifecycle Jul 20, 2026
@alexander-alderman-webb
alexander-alderman-webb marked this pull request as ready for review July 21, 2026 07:39
@alexander-alderman-webb
alexander-alderman-webb requested a review from a team as a code owner July 21, 2026 07:39
Comment thread tests/integrations/anthropic/test_anthropic.py Outdated

@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 463759f. Configure here.

Comment thread tests/integrations/anthropic/test_anthropic.py Outdated
@github-actions

Copy link
Copy Markdown
Contributor

Codecov Results 📊

91914 passed | ⏭️ 6302 skipped | Total: 98216 | Pass Rate: 93.58% | Execution Time: 311m 19s

📊 Comparison with Base Branch

Metric Change
Total Tests
Passed Tests
Failed Tests
Skipped Tests

✨ No test changes detected

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 2478 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        24007     24010        +3
  Branches      8342      8344        +2
==========================================
+ Hits         21530     21532        +2
- Misses        2477      2478        +1
- Partials      1386      1387        +1

Generated by Codecov Action

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.

3 participants