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

[v18.x backport] cli: add --watch by MoLow · Pull Request #44815 · nodejs/node · GitHub

/ node Public

[v18.x backport] cli: add --watch - #44815

Closed
MoLow wants to merge 4 commits into
nodejs:v18.x-stagingfrom
MoLow:backport-v18-watch-mode
Closed

[v18.x backport] cli: add --watch#44815
MoLow wants to merge 4 commits into
nodejs:v18.x-stagingfrom
MoLow:backport-v18-watch-mode

Conversation

MoLow commented Sep 29, 2022
edited
Loading

Copy link
Copy Markdown
Member

this PR backports:
#44366
#44551
#44621
#44739

the change needed was to skip the test combining --import with --watch since --import is blocked from backporting

This comment was marked as off-topic.

nodejs-github-bot added lib / src Issues and PRs related to general changes in the lib or src directory. needs-ci PRs that need a full CI run. v18.x labels Sep 29, 2022
PR-URL: nodejs#44366
Backport-PR-URL: nodejs#44815
Fixes: nodejs#40429
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
PR-URL: nodejs#44551
Backport-PR-URL: nodejs#44815
Reviewed-By: Kohei Ueno <kohei.ueno119@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Nitzan Uziely <linkgoron@gmail.com>
PR-URL: nodejs#44621
Backport-PR-URL: nodejs#44815
Fixes: nodejs#44655
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
PR-URL: nodejs#44739
Backport-PR-URL: nodejs#44815
Fixes: nodejs#44735
Reviewed-By: Erick Wendel <erick.workspace@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
MoLow force-pushed the backport-v18-watch-mode branch from fce6dc8 to f0f5e4f Compare September 29, 2022 09:31

MoLow commented Sep 29, 2022

Copy link
Copy Markdown
Member Author

CC @nodejs/releasers

MoLow added the request-ci Add this label to start a Jenkins CI on a PR. label Sep 29, 2022
github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Sep 29, 2022

This comment was marked as outdated.

This comment was marked as outdated.

Copy link
Copy Markdown
Collaborator

danielleadams pushed a commit that referenced this pull request Oct 2, 2022
PR-URL: #44366
Backport-PR-URL: #44815
Fixes: #40429
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
danielleadams pushed a commit that referenced this pull request Oct 2, 2022
PR-URL: #44551
Backport-PR-URL: #44815
Reviewed-By: Kohei Ueno <kohei.ueno119@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Nitzan Uziely <linkgoron@gmail.com>
danielleadams pushed a commit that referenced this pull request Oct 2, 2022
PR-URL: #44621
Backport-PR-URL: #44815
Fixes: #44655
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
danielleadams pushed a commit that referenced this pull request Oct 2, 2022
PR-URL: #44739
Backport-PR-URL: #44815
Fixes: #44735
Reviewed-By: Erick Wendel <erick.workspace@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>

Copy link
Copy Markdown
Contributor

Landed in 2e7a17d...36227ed

MoLow deleted the backport-v18-watch-mode branch October 2, 2022 10:52
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

lib / src Issues and PRs related to general changes in the lib or src directory. needs-ci PRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL