-
-
Notifications
You must be signed in to change notification settings - Fork 4.4k
ref(loader): Streamline var usage #47316
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
Conversation
🚨 Warning: This pull request contains Frontend and Backend changes! It's discouraged to make changes to Sentry's Frontend and Backend in a single pull request. The Frontend and Backend are not atomically deployed. If the changes are interdependent of each other, they must be separated into two pull requests and be made forward or backwards compatible, such that the Backend or Frontend can be safely deployed independently. Have questions? Please ask in the |
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #47316 +/- ##
===========================================
+ Coverage 61.37% 80.87% +19.49%
===========================================
Files 4738 4770 +32
Lines 201153 201584 +431
Branches 11542 11557 +15
===========================================
+ Hits 123465 163031 +39566
+ Misses 77433 38297 -39136
- Partials 255 256 +1 |
0b9259e
to
2552ecc
Compare
2552ecc
to
f57c96e
Compare
f57c96e
to
a419f62
Compare
a419f62
to
0d3abad
Compare
0d3abad
to
1aa1404
Compare
Update usage of the passed in variables, so we re-use strings as much as possible.