-
Notifications
You must be signed in to change notification settings - Fork 29.7k
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
meta: update AUTHORS #39874
meta: update AUTHORS #39874
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've checked that none of these are duplicates.
Fast-track has been requested by @Trott. Please 👍 to approve. |
Commit Queue failed- Loading data for nodejs/node/pull/39874 ✔ Done loading data for nodejs/node/pull/39874 ----------------------------------- PR info ------------------------------------ Title meta: update AUTHORS (#39874) ⚠ Could not retrieve the email or name of the PR author's from user's GitHub profile! Branch undefined:actions/authors-update -> nodejs:master Labels meta, fast-track Commits 1 - meta: update AUTHORS Committers 1 - github-actions[bot] PR-URL: https://github.com/nodejs/node/pull/39874 Reviewed-By: Rich Trott Reviewed-By: Michaël Zasso Reviewed-By: Luigi Pinca ------------------------------ Generated metadata ------------------------------ PR-URL: https://github.com/nodejs/node/pull/39874 Reviewed-By: Rich Trott Reviewed-By: Michaël Zasso Reviewed-By: Luigi Pinca -------------------------------------------------------------------------------- ℹ This PR was created on Wed, 25 Aug 2021 04:10:23 GMT ✔ Approvals: 3 ✔ - Rich Trott (@Trott) (TSC): https://github.com/nodejs/node/pull/39874#pullrequestreview-737879844 ✔ - Michaël Zasso (@targos) (TSC): https://github.com/nodejs/node/pull/39874#pullrequestreview-737916995 ✔ - Luigi Pinca (@lpinca): https://github.com/nodejs/node/pull/39874#pullrequestreview-738374612 ℹ This PR is being fast-tracked ✖ GitHub CI is still running ℹ Green GitHub Actions CI is sufficient -------------------------------------------------------------------------------- ✔ Aborted `git node land` session in /home/runner/work/node/node/.ncuhttps://github.com/nodejs/node/actions/runs/1166744069 |
I wonder why GitHub CI isn't running for this. |
GitHub actions cannot be triggered by events that are generated by the default token of another action. The solution is to use |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Closing until #39907 or something like it lands, at which point we'll run the workflow again and see if we have GitHub CI running on the PR or not. |
Refs: #39874 (comment) PR-URL: #39907 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Refs: #39874 (comment) PR-URL: #39907 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Refs: #39874 (comment) PR-URL: #39907 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
If this PR exists, there's presumably new additions to the AUTHORS file. This is an automatically generated PR by the
authors.yml
GitHub Action, which runstools/update-authors.js
and submits a new PR or updates an existing PR.Please note that there might be duplicate entries. If there are, please remove them and add the duplicate emails to .mailmap directly to this PR.