Skip to content

Conversation

@devsnek
Copy link
Member

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines

@devsnekdevsnek added the fast-track PRs that do not need to wait for 48 hours to land. label Sep 12, 2020
@nodejs-github-botnodejs-github-bot added the doc Issues and PRs related to the documentations. label Sep 12, 2020
@mmarchini
Copy link
Contributor

+1 to fast track

@devsnekdevsnek added the commit-queue Add this label to land a pull request using GitHub Actions. label Sep 12, 2020
@github-actionsgithub-actionsbot 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 12, 2020
@github-actions
Copy link
Contributor

Commit Queue failed
- Loading data for nodejs/node/pull/35160 ✔ Done loading data for nodejs/node/pull/35160 ----------------------------------- PR info ------------------------------------ Title meta: update my collab entry (#35160) Author devsnek (@devsnek) Branch devsnek:devsnek-patch-1 -> nodejs:master Labels doc, fast-track Commits 1 - meta: update my collab entry Committers 1 - GitHub PR-URL: https://github.com/nodejs/node/pull/35160 Reviewed-By: Colin Ihrig Reviewed-By: Mary Marchini ------------------------------ Generated metadata ------------------------------ PR-URL: https://github.com/nodejs/node/pull/35160 Reviewed-By: Colin Ihrig Reviewed-By: Mary Marchini -------------------------------------------------------------------------------- ✖ GitHub CI is still running ℹ Doc-only changes ℹ This PR was created on Sat, 12 Sep 2020 00:01:33 GMT ✔ Approvals: 2 ✔ - Colin Ihrig (@cjihrig) (TSC): https://github.com/nodejs/node/pull/35160#pullrequestreview-487180831 ✔ - Mary Marchini (@mmarchini) (TSC): https://github.com/nodejs/node/pull/35160#pullrequestreview-487181981 ℹ This PR is being fast-tracked -------------------------------------------------------------------------------- ✔ Aborted `git node land` session in /home/runner/work/node/node/.ncu

@mmarchini
Copy link
Contributor

Heh, need to check for pending on GH Actions as well :)

@TrottTrottforce-pushed the devsnek-patch-1 branch 2 times, most recently from 56b80c1 to b3c6d28CompareSeptember 12, 2020 02:43
@TrottTrott merged commit b3c6d28 into masterSep 12, 2020
@Trott
Copy link
Member

Landed in b3c6d28

@TrottTrott deleted the devsnek-patch-1 branch September 12, 2020 02:43
PR-URL: #35160 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Mary Marchini <[email protected]> Reviewed-By: Rich Trott <[email protected]>
ruyadorno pushed a commit that referenced this pull request Sep 17, 2020
PR-URL: #35160 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Mary Marchini <[email protected]> Reviewed-By: Rich Trott <[email protected]>
@ruyadornoruyadorno mentioned this pull request Sep 21, 2020
4 tasks
addaleax pushed a commit that referenced this pull request Sep 22, 2020
PR-URL: #35160 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Mary Marchini <[email protected]> Reviewed-By: Rich Trott <[email protected]>
@codebyterecodebytere mentioned this pull request Sep 28, 2020
joesepi pushed a commit to joesepi/node that referenced this pull request Jan 8, 2021
PR-URL: nodejs#35160 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Mary Marchini <[email protected]> Reviewed-By: Rich Trott <[email protected]>
@targostargos removed the commit-queue-failed An error occurred while landing this pull request using GitHub Actions. label Sep 5, 2021
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docIssues and PRs related to the documentations.fast-trackPRs that do not need to wait for 48 hours to land.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@devsnek@mmarchini@Trott@cjihrig@targos@nodejs-github-bot