Skip to content

ref(tests): Replace deprecated enable_tracingwith traces_sample_rate#6077

Merged
sentrivana merged 1 commit intomasterfrom
ivana/remove-enable-tracing
Apr 15, 2026
Merged

ref(tests): Replace deprecated enable_tracingwith traces_sample_rate#6077
sentrivana merged 1 commit intomasterfrom
ivana/remove-enable-tracing

Conversation

@sentrivana
Copy link
Copy Markdown
Contributor

@sentrivana sentrivana commented Apr 15, 2026

  • Replaced deprecated enable_tracing=True with traces_sample_rate=1.0 across 9 test files
  • Preserved tests that specifically test the enable_tracing option itself (test_enable_tracing_deprecated, test_option_enable_tracing, test_celery_trace_propagation_enable_tracing)

Replace deprecated `enable_tracing=True` with `traces_sample_rate=1.0`
in tests that only use it for enabling tracing, preserving tests that
specifically test the `enable_tracing` option itself.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 15, 2026

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


New Features ✨

  • (ci) Cancel in-progress PR workflows on new commit push by joshuarli in #5994

Bug Fixes 🐛

  • (google_genai) Redact binary data in inline_data and fix multi-part message extraction by ericapisani in #5977

Internal Changes 🔧

  • (pydantic-ai) Remove dead Model.request patch by alexander-alderman-webb in #5956
  • (tests) Replace deprecated enable_tracingwith traces_sample_rate by sentrivana in #6077
  • Set explicit base-branch for codecov action by ericapisani in #5992

🤖 This preview updates automatically when you update the PR.

@sentrivana sentrivana changed the title ref(tests): Replace enable_tracing=True with traces_sample_rate=1.0 ref(tests): Replace deprecated enable_tracingwith traces_sample_rate Apr 15, 2026
@sentrivana sentrivana marked this pull request as ready for review April 15, 2026 10:02
@sentrivana sentrivana requested a review from a team as a code owner April 15, 2026 10:02
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 15, 2026

Codecov Results 📊

13 passed | Total: 13 | Pass Rate: 100% | Execution Time: 7.70s

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 15030 uncovered lines.


Generated by Codecov Action

@sentrivana sentrivana merged commit 5d26c2f into master Apr 15, 2026
161 checks passed
@sentrivana sentrivana deleted the ivana/remove-enable-tracing branch April 15, 2026 11:50
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