ref(boto3): Move crumbs to integration - #7165
Conversation
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 7bd4d0b. Configure here.
Codecov Results 📊✅ 105373 passed | ❌ 1 failed | ⏭️ 6687 skipped | Total: 112061 | Pass Rate: 94.03% | Execution Time: 369m 9s 📊 Comparison with Base Branch
➕ New Tests (1)View new tests
❌ Failed Tests
|
| File | Patch % | Lines |
|---|---|---|
| sentry_sdk/integrations/boto3.py | 100.00% |
Coverage diff
@@ Coverage Diff @@
## main #PR +/-##
==========================================
+ Coverage 90.14% 90.18% +0.04%
==========================================
Files 193 193 —
Lines 25147 25223 +76
Branches 9136 9176 +40
==========================================
+ Hits 22669 22747 +78
- Misses 2478 2476 -2
- Partials 1431 1434 +3Generated by Codecov Action

Description
Set boto3 breadcrumbs in the integration directly, without relying on
maybe_create_breadcrumbs_from_span.Also, add breadcrumb tests to
test_s3.pyand remove themaybe_create_breadcrumbs_from_spanutil now that nothing needs it anymore.Issues
Closes #7067