Skip to content
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

Cleanup GetSystemMetrics #1469

Merged
merged 1 commit into from
Jul 29, 2019
Merged

Cleanup GetSystemMetrics #1469

merged 1 commit into from
Jul 29, 2019

Conversation

hughbe
Copy link
Contributor

@hughbe hughbe commented Jul 25, 2019

Microsoft Reviewers: Open in CodeFlow

@hughbe hughbe requested a review from a team as a code owner July 25, 2019 12:32
@codecov
Copy link

codecov bot commented Jul 25, 2019

Codecov Report

Merging #1469 into master will decrease coverage by 0.03578%.
The diff coverage is 26.47059%.

@@                 Coverage Diff                 @@
##              master       #1469         +/-   ##
===================================================
- Coverage   25.12242%   25.08664%   -0.03578%     
===================================================
  Files            785         771         -14     
  Lines         268553      268936        +383     
  Branches       37959       37962          +3     
===================================================
  Hits           67467       67467                 
- Misses        196347      196731        +384     
+ Partials        4739        4738          -1
Flag Coverage Δ
#Debug 25.08664% <26.47059%> (-0.03579%) ⬇️
#production 25.08664% <26.47059%> (-0.03579%) ⬇️
#test 100% <ø> (ø) ⬆️

@zsd4yr
Copy link
Contributor

zsd4yr commented Jul 25, 2019

Just a few minor tweaks! In your defense, I looked at the old codebase and the order stuff was already present there :) Not sure for how long...

@hughbe
Copy link
Contributor Author

hughbe commented Jul 26, 2019

All fixed

@hughbe
Copy link
Contributor Author

hughbe commented Jul 29, 2019

@RussKie are you happy with this? its likely going to conflict with other PRs (e.g. #1481 and #1508) so i want to be able to fix that

@RussKie
Copy link
Member

RussKie commented Jul 29, 2019

I'm reviewing it right now

@ghost ghost added the 📭 waiting-author-feedback The team requires more information from the author label Jul 29, 2019
@ghost ghost removed the 📭 waiting-author-feedback The team requires more information from the author label Jul 29, 2019
@RussKie RussKie merged commit 14485a8 into dotnet:master Jul 29, 2019
@hughbe hughbe deleted the cleanup-metrics branch July 29, 2019 13:55
@ghost ghost locked as resolved and limited conversation to collaborators Feb 5, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants