Skip to content

report: simplify TriggerNodeReport() - #26174

Merged
cjihrig merged 1 commit into
nodejs:masterfrom
cjihrig:report
Feb 20, 2019
Merged

report: simplify TriggerNodeReport()#26174
cjihrig merged 1 commit into
nodejs:masterfrom
cjihrig:report

Conversation

@cjihrig

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

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-botnodejs-github-bot added the c++ Issues and PRs that require attention from people who are familiar with C++. label Feb 17, 2019
@richardlaurichardlau added the report Issues and PRs related to process.report. label Feb 17, 2019
@danbev

Copy link
Copy Markdown
Contributor

@cjihrig

cjihrig commented Feb 18, 2019

Copy link
Copy Markdown
ContributorAuthor

Resume CI: https://ci.nodejs.org/job/node-test-pull-request/20873/

EDIT: CI was green.

PR-URL: nodejs#26174
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
@cjihrig
cjihrig merged commit 1dc22bf into nodejs:masterFeb 20, 2019
@cjihrig
cjihrig deleted the report branch February 20, 2019 13:45
addaleax pushed a commit that referenced this pull request Feb 21, 2019
PR-URL: #26174
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
@BridgeARBridgeAR mentioned this pull request Feb 26, 2019
rvagg pushed a commit that referenced this pull request Feb 28, 2019
PR-URL: #26174
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
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.

6 participants

@cjihrig@nodejs-github-bot@danbev@jasnell@addaleax@richardlau