Skip to content

test: add logging to statwatcher test - #28270

Merged
Trott merged 1 commit into
nodejs:masterfrom
Trott:moar-logging-statwatcher
Jun 20, 2019
Merged

test: add logging to statwatcher test#28270
Trott merged 1 commit into
nodejs:masterfrom
Trott:moar-logging-statwatcher

Conversation

@Trott

Copy link
Copy Markdown
Member

Refs: #21425 (comment)

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

@nodejs-github-botnodejs-github-bot added async_hooks Issues and PRs related to the async hooks subsystem. test Issues and PRs related to the tests. labels Jun 17, 2019
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@TrottTrott added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Jun 19, 2019
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Refs: nodejs#21425 (comment)
PR-URL: nodejs#28270
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Refael Ackermann (רפאל פלחי) <refack@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
@Trott
Trottforce-pushed the moar-logging-statwatcher branch from b727277 to f28b1edCompareJune 20, 2019 04:12
@Trott

Copy link
Copy Markdown
MemberAuthor

Landed in f28b1ed

@Trott
Trott merged commit f28b1ed into nodejs:masterJun 20, 2019
targos pushed a commit that referenced this pull request Jul 2, 2019
Refs: #21425 (comment)
PR-URL: #28270
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Refael Ackermann (רפאל פלחי) <refack@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
@targostargos mentioned this pull request Jul 2, 2019
@Trott
Trott deleted the moar-logging-statwatcher branch January 13, 2022 22:51
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

async_hooksIssues and PRs related to the async hooks subsystem.author readyPRs that have at least one approval, no pending requests for changes, and a CI started.testIssues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@Trott@nodejs-github-bot@refack@addaleax@BridgeAR