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

test: remove useless WPT init scripts by KhafraDev · Pull Request #47221 · nodejs/node · GitHub

/ node Public

test: remove useless WPT init scripts - #47221

Closed
KhafraDev wants to merge 1 commit into
nodejs:mainfrom
KhafraDev:useless-wpt-initscripts
Closed

test: remove useless WPT init scripts#47221
KhafraDev wants to merge 1 commit into
nodejs:mainfrom
KhafraDev:useless-wpt-initscripts

Conversation

Copy link
Copy Markdown
Member

File is now a global so the init scripts are no longer needed.

Refs: #47153 (review)

nodejs-github-bot added needs-ci PRs that need a full CI run. test Issues and PRs related to the tests. labels Mar 22, 2023
panva added dont-land-on-v14.x request-ci Add this label to start a Jenkins CI on a PR. labels Mar 22, 2023
panva added the author ready PRs that have at least one approval, no outstanding review comments, and a CI started. label Mar 22, 2023
github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Mar 22, 2023

Copy link
Copy Markdown
Collaborator

Copy link
Copy Markdown
Collaborator

Copy link
Copy Markdown
Collaborator

Copy link
Copy Markdown
Collaborator

panva commented Mar 28, 2023

Copy link
Copy Markdown
Member

Landed in 9dfd039

panva closed this Mar 28, 2023
panva pushed a commit that referenced this pull request Mar 28, 2023
PR-URL: #47221
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Filip Skokan <panva.ip@gmail.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
KhafraDev deleted the useless-wpt-initscripts branch March 28, 2023 14:39
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 Issues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants


Back | FazBrowse Home | New Git URL