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

test_runner: apply `runOnly` on suites by MoLow · Pull Request #48279 · nodejs/node · GitHub

/ node Public

test_runner: apply runOnly on suites - #48279

Closed
MoLow wants to merge 1 commit into
nodejs:mainfrom
MoLow:fix-suite-children-only
Closed

test_runner: apply runOnly on suites#48279
MoLow wants to merge 1 commit into
nodejs:mainfrom
MoLow:fix-suite-children-only

Conversation

MoLow commented Jun 1, 2023

Copy link
Copy Markdown
Member

Fixes: #47937

MoLow requested review from benjamingr and cjihrig June 1, 2023 10:06

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/test_runner

nodejs-github-bot added needs-ci PRs that need a full CI run. test_runner Issues and PRs related to the test runner subsystem. labels Jun 1, 2023

MoLow commented Jun 3, 2023

Copy link
Copy Markdown
Member Author

ping @nodejs/test_runner can someone take a look?

MoLow added author ready PRs that have at least one approval, no outstanding review comments, and a CI started. request-ci Add this label to start a Jenkins CI on a PR. labels Jun 4, 2023
github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jun 4, 2023

Copy link
Copy Markdown
Collaborator

MoLow added a commit that referenced this pull request Jun 4, 2023
PR-URL: #48279
Fixes: #47937
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>

MoLow commented Jun 4, 2023

Copy link
Copy Markdown
Member Author

Landed in 5c27cc2

MoLow closed this Jun 4, 2023
MoLow deleted the fix-suite-children-only branch June 4, 2023 21:06
targos pushed a commit that referenced this pull request Jun 5, 2023
PR-URL: #48279
Fixes: #47937
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
targos mentioned this pull request Jun 5, 2023
danielleadams pushed a commit that referenced this pull request Jul 6, 2023
PR-URL: #48279
Fixes: #47937
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
MoLow added a commit to MoLow/node that referenced this pull request Jul 6, 2023
PR-URL: nodejs#48279
Fixes: nodejs#47937
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Ceres6 pushed a commit to Ceres6/node that referenced this pull request Aug 14, 2023
PR-URL: nodejs#48279
Fixes: nodejs#47937
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Ceres6 pushed a commit to Ceres6/node that referenced this pull request Aug 14, 2023
PR-URL: nodejs#48279
Fixes: nodejs#47937
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
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

author ready PRs that have at least one approval, no outstanding review comments, and a CI started. needs-ci PRs that need a full CI run. test_runner Issues and PRs related to the test runner subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

it.only inside describe.only run all other its as well

6 participants


Back | FazBrowse Home | New Git URL