Skip to content

fix: Random formatting, comment cleanup - #8256

Merged
sentrivana merged 1 commit into
major/3.0from
ivana/major/deslop-results
Nov 12, 2026
Merged

sentrivana merged 1 commit into
major/3.0from
ivana/major/deslop-results

Conversation

@sentrivana

Copy link
Copy Markdown
Collaborator

Ran a deslop tool on the SDK. Didn't work that great. This is some actionable stuff that it did.

@sentrivana sentrivana changed the title fix: Random formatting, comment fixes fix: Random formatting, comment cleanup Nov 12, 2026
@sentrivana
sentrivana marked this pull request as ready for review November 12, 2026 10:04
@sentrivana
sentrivana requested a review from a team as a code owner November 12, 2026 10:04
@sentrivana
sentrivana merged commit a521dc4 into major/3.0 Nov 12, 2026
152 checks passed
@sentrivana
sentrivana deleted the ivana/major/deslop-results branch November 12, 2026 10:05
@github-actions

Copy link
Copy Markdown
Contributor

Coverage Results 📊

✅ 58786 passed | ⏭️ 3071 skipped | Total: 61857 | Pass Rate: 95.04% | Execution Time: 165m 51s

📊 Comparison with Base Branch

Metric Change
Total Tests 📉 -85122
Passed Tests 📉 -80380
Failed Tests —
Skipped Tests 📉 -4742

All tests are passing successfully.

✅ Patch coverage is 100.00% (no changed executable lines found; target 80%).
Project statement coverage is 90.45% (up 0.21 percentage points from base (8abf619) to head (698fa0a)).

Files with missing lines (2)
File Coverage % Lines
sentry_sdk/integrations/aiomysql.py 94.16% ⚠️ 8 Missing and 17 partials
sentry_sdk/integrations/pydantic_ai/spans/invoke_agent.py 100.00% ⚠️ 9 partials
Coverage diff
@@            Coverage Diff             @@
##        master     #8256       +/-##
==========================================
+ Coverage    90.24%    90.45%    +0.21%
==========================================
  Files          203       199        -4
  Tracked lines     27235     21786     -5449
  Branches     10200      7582     -2618
==========================================
- Hits         24577     19705     -4872
+ Misses        2658      2081      -577
+ Partials      1555      1254      -301

Generated by Coverage 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.

2 participants