Skip to content

Conversation

@cjihrig
Copy link
Contributor

Prior to this change, user and system CPU were the same values.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines

@nodejs-github-bot
Copy link
Collaborator

@cjihrig sadly an error occured when I tried to trigger a build :(

@nodejs-github-botnodejs-github-bot added the c++ Issues and PRs that require attention from people who are familiar with C++. label Feb 24, 2019
@richardlaurichardlau added the report Issues and PRs related to process.report. label Feb 24, 2019
PR-URL: nodejs#26286 Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: James M Snell <[email protected]>
@cjihrig
Copy link
ContributorAuthor

@cjihrigcjihrig merged commit 4255190 into nodejs:masterFeb 27, 2019
@cjihrigcjihrig deleted the report-cpu branch February 27, 2019 00:59
addaleax pushed a commit that referenced this pull request Mar 1, 2019
PR-URL: #26286 Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: James M Snell <[email protected]>
@BridgeARBridgeAR mentioned this pull request Mar 4, 2019
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c++Issues and PRs that require attention from people who are familiar with C++.reportIssues and PRs related to process.report.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@cjihrig@nodejs-github-bot@jasnell@addaleax@lpinca@targos@richardlau