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

meta: update AUTHORS #39874

Closed
wants to merge 1 commit into from
Closed

meta: update AUTHORS #39874

wants to merge 1 commit into from

Conversation

github-actions[bot]
Copy link
Contributor

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 runs tools/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.

@github-actions github-actions bot added the meta Issues and PRs related to the general management of the project. label Aug 25, 2021
@Trott
Copy link
Member

Trott commented Aug 25, 2021

@targos @Mesteery @bnb That seems to have done the trick, third time's the charm, and all that?

Copy link
Member

@Trott Trott left a 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.

@Trott Trott added the fast-track PRs that do not need to wait for 48 hours to land. label Aug 25, 2021
@github-actions
Copy link
Contributor Author

Fast-track has been requested by @Trott. Please 👍 to approve.

@Trott Trott added the commit-queue Add this label to land a pull request using GitHub Actions. label Aug 25, 2021
@github-actions github-actions bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Aug 25, 2021
@github-actions
Copy link
Contributor Author

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/.ncu
https://github.com/nodejs/node/actions/runs/1166744069

@github-actions github-actions bot added the commit-queue-failed An error occurred while landing this pull request using GitHub Actions. label Aug 25, 2021
@Trott
Copy link
Member

Trott commented Aug 25, 2021

I wonder why GitHub CI isn't running for this.

@targos
Copy link
Member

targos commented Aug 25, 2021

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 ${{ secrets.GH_USER_TOKEN }}

Copy link
Member

@mhdawson mhdawson left a comment

Choose a reason for hiding this comment

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

LGTM

Trott added a commit to Trott/io.js that referenced this pull request Aug 27, 2021
@Trott
Copy link
Member

Trott commented Aug 27, 2021

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.

@Trott Trott closed this Aug 27, 2021
Trott added a commit to Trott/io.js that referenced this pull request Aug 29, 2021
nodejs-github-bot pushed a commit that referenced this pull request Aug 30, 2021
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>
@targos targos removed the commit-queue-failed An error occurred while landing this pull request using GitHub Actions. label Sep 5, 2021
targos pushed a commit that referenced this pull request Sep 6, 2021
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>
targos pushed a commit that referenced this pull request Sep 6, 2021
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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fast-track PRs that do not need to wait for 48 hours to land. meta Issues and PRs related to the general management of the project.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants