-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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(rules): Fix environment fetch #73665
Conversation
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.
50b779a
to
b26fff3
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #73665 +/- ##
===========================================
- Coverage 88.21% 78.07% -10.15%
===========================================
Files 2928 6652 +3724
Lines 182133 297331 +115198
Branches 32473 51166 +18693
===========================================
+ Hits 160677 232151 +71474
- Misses 15519 58870 +43351
- Partials 5937 6310 +373
|
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.
Copy/paste bug where the
.get()
was on the wrong variable for the environment and was always returning None so if an environment was present on the rule and then was removed, it wouldn't be added to the confirmation notification. See https://github.com/getsentry/sentry/pull/66847/files#r1662799957