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
[sentry_lumberdash] Support latest sentry version and add tests #75
Conversation
Codecov Report
@@ Coverage Diff @@
## master #75 +/- ##
===========================================
+ Coverage 36.45% 48.45% +11.99%
===========================================
Files 6 6
Lines 96 97 +1
===========================================
+ Hits 35 47 +12
+ Misses 61 50 -11
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
HubAdapter forwards to Sentrys static API. This makes it more resilient. This way you could init lumberdash before Sentry or after Sentry. This doesn't work with Sentry.currentHub. CurrentHub is a NoOp Hub until Sentry is initialized. This means you couldn't even init lumberdash inside Sentrys initApp callback.
Docker pull failed but the build should be green if you restart it. Instead of docker you could also use the GitHub Dart Action: https://github.com/dart-lang/setup-dart |
@fabiomcarneiro or @fwagner do you mind taking a look at this? |
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 . Thank you.
Thanks @ueman. New version is available on pub.dev: https://pub.dev/packages/sentry_lumberdash/versions/3.1.0. |
This PR updates sentry_lumberdash to the latest Sentry version.
I also added tests and it should now have complete coverage.
I've also improved the documentation and various links.