FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

test: remove custom AsyncHooksTestConfiguration by maclover7 · Pull Request #22008 · nodejs/node · GitHub

/ node Public

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

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`

Copy link
Copy Markdown
Collaborator

nodejs-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

Copy link
Copy Markdown
Contributor Author

Trott commented Jul 29, 2018
edited by maclover7
Loading

Copy link
Copy Markdown
Member

Resume Build: https://ci.nodejs.org/job/node-test-pull-request/16053/ (edit @maclover7: ✔️)

maclover7 added the author ready PRs that have at least one approval, no outstanding review comments, and a CI started. label Jul 29, 2018

Copy link
Copy Markdown
Contributor Author

Landed in 3a056c0

maclover7 closed this Jul 31, 2018
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>
targos mentioned this pull request Jul 31, 2018
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

async_hooks Issues and PRs related to the async hooks subsystem. author ready PRs that have at least one approval, no outstanding review comments, and a CI started. test Issues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL