Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: mark test-fs-watch-non-recursive flaky on Windows #40916

Merged

Conversation

lpinca
Copy link
Member

@lpinca lpinca commented Nov 21, 2021

Refs: #40728

@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 Nov 21, 2021
@nodejs-github-bot

This comment has been minimized.

@nodejs-github-bot

This comment has been minimized.

@nodejs-github-bot
Copy link
Collaborator

CI: https://ci.nodejs.org/job/node-test-pull-request/41044/

@Flarna Flarna added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. and removed needs-ci PRs that need a full CI run. labels Nov 22, 2021
@lpinca lpinca added the commit-queue Add this label to land a pull request using GitHub Actions. label Nov 23, 2021
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Nov 23, 2021
@nodejs-github-bot
Copy link
Collaborator

Commit Queue failed
- Loading data for nodejs/node/pull/40916
✔  Done loading data for nodejs/node/pull/40916
----------------------------------- PR info ------------------------------------
Title      test: mark test-fs-watch-non-recursive flaky on Windows (#40916)
   ⚠  Could not retrieve the email or name of the PR author's from user's GitHub profile!
Branch     lpinca:mark/test-fs-watch-non-recursive-flaky -> nodejs:master
Labels     test, author ready
Commits    1
 - test: mark test-fs-watch-non-recursive flaky on Windows
Committers 1
 - Luigi Pinca 
PR-URL: https://github.com/nodejs/node/pull/40916
Refs: https://github.com/nodejs/node/issues/40728
Reviewed-By: Gerhard Stöbich 
Reviewed-By: Michaël Zasso 
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/40916
Refs: https://github.com/nodejs/node/issues/40728
Reviewed-By: Gerhard Stöbich 
Reviewed-By: Michaël Zasso 
--------------------------------------------------------------------------------
   ℹ  This PR was created on Sun, 21 Nov 2021 20:04:09 GMT
   ✔  Approvals: 2
   ✔  - Gerhard Stöbich (@Flarna): https://github.com/nodejs/node/pull/40916#pullrequestreview-812234955
   ✔  - Michaël Zasso (@targos) (TSC): https://github.com/nodejs/node/pull/40916#pullrequestreview-812236618
   ✖  This PR needs to wait 5 more hours to land
   ✔  Last GitHub Actions successful
   ℹ  Last Full PR CI on 2021-11-22T13:55:43Z: https://ci.nodejs.org/job/node-test-pull-request/41044/
- Querying data for job/node-test-pull-request/41044/
   ✔  Last Jenkins CI successful
--------------------------------------------------------------------------------
   ✔  Aborted `git node land` session in /home/runner/work/node/node/.ncu
https://github.com/nodejs/node/actions/runs/1495305752

@nodejs-github-bot nodejs-github-bot added the commit-queue-failed An error occurred while landing this pull request using GitHub Actions. label Nov 23, 2021
@Flarna Flarna added the commit-queue Add this label to land a pull request using GitHub Actions. label Nov 23, 2021
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Nov 23, 2021
@nodejs-github-bot nodejs-github-bot merged commit 2a871df into nodejs:master Nov 23, 2021
@nodejs-github-bot
Copy link
Collaborator

Landed in 2a871df

@lpinca lpinca deleted the mark/test-fs-watch-non-recursive-flaky branch November 25, 2021 10:31
targos pushed a commit that referenced this pull request Nov 26, 2021
Refs: #40728

PR-URL: #40916
Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
danielleadams pushed a commit that referenced this pull request Jan 30, 2022
Refs: #40728

PR-URL: #40916
Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
danielleadams pushed a commit that referenced this pull request Feb 1, 2022
Refs: #40728

PR-URL: #40916
Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
@danielleadams danielleadams mentioned this pull request Feb 1, 2022
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 pending requests for changes, and a CI started. commit-queue-failed An error occurred while landing this pull request using GitHub Actions. test Issues and PRs related to the tests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants