Skip to content

ref: Remove should_send_default_pii helpers from client and scope - #7892

Merged
ericapisani merged 1 commit into
py-2932-remove-map-send-default-piifrom
py-2932-remove-should-send-default-pii
Oct 6, 2026
Merged

ericapisani merged 1 commit into
py-2932-remove-map-send-default-piifrom
py-2932-remove-should-send-default-pii

Conversation

@ericapisani

Copy link
Copy Markdown
Member

Drop BaseClient.should_send_default_pii, _Client.should_send_default_pii and the module-level scope.should_send_default_pii shortcut.

Fixes PY-2932
Fixes #7882

Drop `BaseClient.should_send_default_pii`, `_Client.should_send_default_pii`
and the module-level `scope.should_send_default_pii` shortcut.

Fixes PY-2932
Fixes #7882
@ericapisani
ericapisani requested a review from a team as a code owner October 6, 2026 15:34
@linear-code

linear-code Bot commented Oct 6, 2026

Copy link
Copy Markdown

PY-2932

@github-actions

github-actions Bot commented Oct 6, 2026

Copy link
Copy Markdown
Contributor

Codecov Results 📊

✅ 55379 passed | ⏭️ 2727 skipped | Total: 58106 | Pass Rate: 95.31% | Execution Time: 150m 25s

📊 Comparison with Base Branch

Metric Change
Total Tests 📉 -82279
Passed Tests 📉 -77783
Failed Tests 📉 -1
Skipped Tests 📉 -4495

➖ Removed Tests (1)

View removed tests
  • test_cache_spans_decorator[True]
    • File: tests.integrations.django.test_cache_module

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 2061 uncovered lines.
❌ Project coverage is 90.03%. Comparing base (23d3ae0) to head (2d6cd1d).

Coverage diff
@@            Coverage Diff             @@
##        master       #PR       +/-##
==========================================
- Coverage    90.38%    90.03%    -0.35%
==========================================
  Files          202       185       -17
  Lines        26552     20672     -5880
  Branches      9858      7116     -2742
==========================================
+ Hits         23997     18611     -5386
- Misses        2555      2061      -494
- Partials      1487      1216      -271

Generated by Codecov Action

@ericapisani
ericapisani merged commit 59aa0c4 into py-2932-remove-map-send-default-pii Oct 6, 2026
147 checks passed
@ericapisani
ericapisani deleted the py-2932-remove-should-send-default-pii branch October 6, 2026 15:42
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