Skip to content

Conversation

@lpinca
Copy link
Member

This reverts commit 717e233.

Refs: fef7927cc3a7

@nodejs-github-botnodejs-github-bot added needs-ci PRs that need a full CI run. test Issues and PRs related to the tests. labels Sep 27, 2023
@aduh95aduh95 added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. commit-queue Add this label to land a pull request using GitHub Actions. and removed needs-ci PRs that need a full CI run. labels Sep 28, 2023
@nodejs-github-botnodejs-github-bot added commit-queue-failed An error occurred while landing this pull request using GitHub Actions. and removed commit-queue Add this label to land a pull request using GitHub Actions. labels Sep 29, 2023
@nodejs-github-bot
Copy link
Collaborator

Commit Queue failed
- Loading data for nodejs/node/pull/49905 ✔ Done loading data for nodejs/node/pull/49905 ----------------------------------- PR info ------------------------------------ Title Revert "test: mark test-runner-output as flaky" (#49905) ⚠ Could not retrieve the email or name of the PR author's from user's GitHub profile! Branch lpinca:revert/717e233cd956 -> nodejs:main Labels test, author ready Commits 1 - Revert "test: mark test-runner-output as flaky" Committers 1 - Luigi Pinca PR-URL: https://github.com/nodejs/node/pull/49905 Refs: https://github.com/nodejs/node/commit/fef7927cc3a7 Reviewed-By: Colin Ihrig Reviewed-By: Moshe Atlow ------------------------------ Generated metadata ------------------------------ PR-URL: https://github.com/nodejs/node/pull/49905 Refs: https://github.com/nodejs/node/commit/fef7927cc3a7 Reviewed-By: Colin Ihrig Reviewed-By: Moshe Atlow -------------------------------------------------------------------------------- ℹ This PR was created on Wed, 27 Sep 2023 17:41:11 GMT ✔ Approvals: 2 ✔ - Colin Ihrig (@cjihrig) (TSC): https://github.com/nodejs/node/pull/49905#pullrequestreview-1647260204 ✔ - Moshe Atlow (@MoLow) (TSC): https://github.com/nodejs/node/pull/49905#pullrequestreview-1647379240 ✔ Last GitHub CI successful ✘ No Jenkins CI runs detected -------------------------------------------------------------------------------- ✔ Aborted `git node land` session in /home/runner/work/node/node/.ncu
https://github.com/nodejs/node/actions/runs/6355169313

@lpincalpinca added the request-ci Add this label to start a Jenkins CI on a PR. label Sep 29, 2023
@github-actionsgithub-actionsbot removed the request-ci Add this label to start a Jenkins CI on a PR. label Sep 29, 2023
@nodejs-github-bot
Copy link
Collaborator

Copy link
Member

@mhdawsonmhdawson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@lpincalpinca added commit-queue Add this label to land a pull request using GitHub Actions. and removed commit-queue-failed An error occurred while landing this pull request using GitHub Actions. labels Sep 30, 2023
@nodejs-github-botnodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Sep 30, 2023
@nodejs-github-botnodejs-github-bot merged commit f05d6d0 into nodejs:mainSep 30, 2023
@nodejs-github-bot
Copy link
Collaborator

Landed in f05d6d0

@lpincalpinca deleted the revert/717e233cd956 branch September 30, 2023 15:45
GeoffreyBooth pushed a commit to GeoffreyBooth/node that referenced this pull request Oct 1, 2023
This reverts commit 717e233. Refs: nodejs@fef7927cc3a7 PR-URL: nodejs#49905 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Michael Dawson <[email protected]>
alexfernandez pushed a commit to alexfernandez/node that referenced this pull request Nov 1, 2023
This reverts commit 717e233. Refs: nodejs@fef7927cc3a7 PR-URL: nodejs#49905 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Michael Dawson <[email protected]>
targos pushed a commit that referenced this pull request Nov 11, 2023
This reverts commit 717e233. Refs: fef7927cc3a7 PR-URL: #49905 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Michael Dawson <[email protected]>
@targostargos mentioned this pull request Nov 12, 2023
debadree25 pushed a commit to debadree25/node that referenced this pull request Apr 15, 2024
This reverts commit 717e233. Refs: nodejs@fef7927cc3a7 PR-URL: nodejs#49905 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Michael Dawson <[email protected]>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.testIssues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@lpinca@nodejs-github-bot@cjihrig@MoLow@mhdawson@targos@aduh95