Skip to content

Conversation

@avivkeller
Copy link
Member

This PR fixes an issue in the tests, which cause similar CLI flags to get mixed up when testing for them in the documentation

(Ref: #52593)

Before this change, the following would both get treated as --my-foo (in the tests, not the actual CLI parsing)

--my-foo
--my-foo-bar

@nodejs-github-bot nodejs-github-bot added needs-ci PRs that need a full CI run. test Issues and PRs related to the tests. labels Apr 22, 2024
@cola119 cola119 added the request-ci Add this label to start a Jenkins CI on a PR. label Apr 23, 2024
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Apr 23, 2024
@nodejs-github-bot
Copy link
Collaborator

@avivkeller
Copy link
Member Author

The CI failed due to a disconnected runner/agent, should we re-run it, or leave it?

@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@cola119
Copy link
Member

cola119 commented Apr 24, 2024

The CI failed due to a disconnected runner/agent, should we re-run it, or leave it?

I restarted the CI, let's wait until everything turns green.

@MoLow
Copy link
Member

MoLow commented Apr 24, 2024

@cola119 please use the "resume build" option next time to avoid running a full CI

@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@MoLow MoLow added the commit-queue Add this label to land a pull request using GitHub Actions. label Apr 24, 2024
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Apr 24, 2024
@nodejs-github-bot nodejs-github-bot merged commit fa62544 into nodejs:main Apr 24, 2024
@nodejs-github-bot
Copy link
Collaborator

Landed in fa62544

@avivkeller avivkeller deleted the patch-13 branch April 24, 2024 19:17
aduh95 pushed a commit that referenced this pull request Apr 29, 2024
PR-URL: #52643
Reviewed-By: Kohei Ueno <kohei.ueno119@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
@marco-ippolito marco-ippolito added the dont-land-on-v20.x PRs that should not land on the v20.x-staging branch and should not be released in v20.x. label Jun 17, 2024
bmeck pushed a commit to bmeck/node that referenced this pull request Jun 22, 2024
PR-URL: nodejs#52643
Reviewed-By: Kohei Ueno <kohei.ueno119@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dont-land-on-v20.x PRs that should not land on the v20.x-staging branch and should not be released in v20.x. 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.

6 participants