-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Split the cloud output so metric pushing is separate #1905
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1905 +/- ##
==========================================
+ Coverage 71.19% 71.22% +0.03%
==========================================
Files 183 184 +1
Lines 14326 14337 +11
==========================================
+ Hits 10199 10212 +13
+ Misses 3501 3499 -2
Partials 626 626
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
39c7df9
to
04028f8
Compare
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, but why was this done? So that we could test/mock the metric pushing?
b9f9487
to
005c657
Compare
Yes, though mostly so we can push metrics without initializing a full-blown |
Continuation of #1857 and #1874