-
-
Notifications
You must be signed in to change notification settings - Fork 206
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
Data from Scope in options should be applied on each request #1270
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1270 +/- ##
==========================================
+ Coverage 81.83% 82.55% +0.71%
==========================================
Files 212 213 +1
Lines 7075 7089 +14
Branches 1396 1399 +3
==========================================
+ Hits 5790 5852 +62
+ Misses 845 801 -44
+ Partials 440 436 -4
Continue to review full report at Codecov.
|
minor changes re exception handling in tests. but the rest LGTM (considering my limited knowledge of the codebase) |
1 similar comment
minor changes re exception handling in tests. but the rest LGTM (considering my limited knowledge of the codebase) |
Since there are no more open comments about this PR I believe it's safe to say that it's read to Merge @bruno-garcia |
The PR applies the configure scopes stored on Sentry options into the hub once and reapplies it if the hud changed.
Close #968