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
Update Sentry in SentryLumberdash #78
Conversation
Codecov Report
@@ Coverage Diff @@
## master #78 +/- ##
==========================================
+ Coverage 41.17% 48.45% +7.27%
==========================================
Files 5 6 +1
Lines 85 97 +12
==========================================
+ Hits 35 47 +12
Misses 50 50
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
@@ -15,8 +15,7 @@ environment: | |||
dependencies: | |||
lumberdash: ^3.0.0 | |||
meta: ^1.3.0 | |||
sentry: ^5.0.0 | |||
sentry: ^6.0.0-0 |
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.
This makes it compatible with the current Sentry beta up to but exclusive version 7.x.x.
|
||
dev_dependencies: | ||
test: ^1.16.5 | ||
mockito: ^5.0.0 |
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.
Mockito isn't used anymore
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
This updates Sentry to version 6 which is currently in beta.
The parts of Sentry which are used in this project are stable and there aren't any planned breaking changes left (I'm one of the maintainers).
So this should be safe to publish. However, I'll leave it up to you if you want to do that.
cc @fwagner @fabiomcarneiro