Skip to content

test: remove custom AsyncHooksTestConfiguration - #22008

Closed
maclover7 wants to merge 1 commit into
nodejs:masterfrom
maclover7:jm-simplify-test
Closed

test: remove custom AsyncHooksTestConfiguration#22008
maclover7 wants to merge 1 commit into
nodejs:masterfrom
maclover7:jm-simplify-test

Conversation

@maclover7

Copy link
Copy Markdown
Contributor

Has the same behavior as ParallelTestConfiguration, so switch to using that :)

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

Has the same behavior as `ParallelTestConfiguration`
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@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 Jul 28, 2018
@maclover7

Copy link
Copy Markdown
ContributorAuthor

@Trott

Trott commented Jul 29, 2018

Copy link
Copy Markdown
Member

@maclover7maclover7 added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Jul 29, 2018
@maclover7

Copy link
Copy Markdown
ContributorAuthor

Landed in 3a056c0

@maclover7
maclover7 deleted the jm-simplify-test branch July 31, 2018 03:51
maclover7 added a commit that referenced this pull request Jul 31, 2018
Has the same behavior as `ParallelTestConfiguration`
PR-URL: #22008
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
targos pushed a commit that referenced this pull request Jul 31, 2018
Has the same behavior as `ParallelTestConfiguration`
PR-URL: #22008
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
@targostargos mentioned this pull request Jul 31, 2018
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.

4 participants

@maclover7@nodejs-github-bot@Trott@BridgeAR