FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

chore(bottle): Remove transaction test by sentrivana · Pull Request #7272 · getsentry/sentry-python · GitHub

chore(bottle): Remove transaction test - #7272

Open
sentrivana wants to merge 4 commits into
major/3.0from
ivana/major/remove-transactions-from-bottle
Open

chore(bottle): Remove transaction test#7272
sentrivana wants to merge 4 commits into
major/3.0from
ivana/major/remove-transactions-from-bottle

Conversation

Copy link
Copy Markdown
Contributor

This should not be needed anymore.

sentrivana force-pushed the ivana/major/remove-transactions-from-trytond-wsgi branch 3 times, most recently from 832e83e to 46fe396 Compare August 27, 2026 12:08
sentrivana changed the base branch from ivana/major/remove-transactions-from-trytond-wsgi to major/3.0 August 27, 2026 12:09
sentrivana force-pushed the ivana/major/remove-transactions-from-bottle branch from 4e951ab to 0633083 Compare August 27, 2026 12:10
sentrivana marked this pull request as ready for review August 27, 2026 12:13
sentrivana requested a review from a team as a code owner August 27, 2026 12:13

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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

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 0633083. Configure here.

sentrivana marked this pull request as draft August 27, 2026 12:23

github-actions Bot commented Aug 27, 2026
edited
Loading

Copy link
Copy Markdown
Contributor

Codecov Results 📊

111237 passed | ⏭️ 5428 skipped | Total: 116665 | Pass Rate: 95.35% | Execution Time: 379m 40s

📊 Comparison with Base Branch

Metric Change
Total Tests 📈 +11264
Passed Tests 📈 +12315
Failed Tests
Skipped Tests 📉 -1051

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 2235 uncovered lines.
✅ Project coverage is 90.61%. Comparing base (base) to head (head).

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    90.23%    90.61%    +0.38%
==========================================
  Files          193       185        -8
  Lines        25482     23811     -1671
  Branches      9384      8788      -596
==========================================
+ Hits         22994     21576     -1418
- Misses        2488      2235      -253
- Partials      1431      1362       -69

Generated by Codecov Action

sentrivana changed the title chore(bottle): Remove transaction code chore(bottle): Remove transaction test Aug 28, 2026
sentrivana marked this pull request as ready for review August 28, 2026 08:51
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL