New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix: set status and associate events with transactions #1426
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1426 +/- ##
============================================
+ Coverage 75.80% 75.84% +0.04%
- Complexity 1866 1870 +4
============================================
Files 185 186 +1
Lines 6389 6405 +16
Branches 637 639 +2
============================================
+ Hits 4843 4858 +15
Misses 1258 1258
- Partials 288 289 +1
Continue to review full report at Codecov.
|
sentry-test-support/src/main/kotlin/io/sentry/test/assertions.kt
Outdated
Show resolved
Hide resolved
Co-authored-by: Manoel Aranda Neto <5731772+marandaneto@users.noreply.github.com>
Co-authored-by: Manoel Aranda Neto <5731772+marandaneto@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
CI isnt happy though
…ntry/sentry-java into gh-1351-transaction-advice
📜 Description
Set status when exception is thrown during transaction execution, and associate exceptions with transactions.
💡 Motivation and Context
#1351
💚 How did you test it?
Unit & Integration tests
📝 Checklist