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 #39855

Closed
wants to merge 1 commit into from
Closed

meta: update AUTHORS #39855

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 24, 2021
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 checked that these aren't duplicates. I think this can land.

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

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

@bnb
Copy link
Contributor

bnb commented Aug 24, 2021

yay! 🎉

@Lxxyx Lxxyx added the commit-queue Add this label to land a pull request using GitHub Actions. label Aug 24, 2021
@github-actions github-actions 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 Aug 24, 2021
@github-actions
Copy link
Contributor Author

Commit Queue failed
- Loading data for nodejs/node/pull/39855
✔  Done loading data for nodejs/node/pull/39855
----------------------------------- PR info ------------------------------------
Title      meta: update AUTHORS (#39855)
   ⚠  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
 - Create or Update Pull Request Action 
PR-URL: https://github.com/nodejs/node/pull/39855
Reviewed-By: Rich Trott 
Reviewed-By: Zijian Liu 
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/39855
Reviewed-By: Rich Trott 
Reviewed-By: Zijian Liu 
--------------------------------------------------------------------------------
   ℹ  This PR was created on Tue, 24 Aug 2021 00:14:59 GMT
   ✔  Approvals: 2
   ✔  - Rich Trott (@Trott) (TSC): https://github.com/nodejs/node/pull/39855#pullrequestreview-736667359
   ✔  - Zijian Liu (@Lxxyx): https://github.com/nodejs/node/pull/39855#pullrequestreview-736870593
   ℹ  This PR is being fast-tracked
   ⚠  GitHub cannot link the author of 'meta: update AUTHORS' to their GitHub account.
   ⚠  Please suggest them to take a look at https://github.com/nodejs/node/blob/99b1ada/doc/guides/contributing/pull-requests.md#step-1-fork
   ✖  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/1161618723

Mesteery added a commit to Mesteery/node that referenced this pull request Aug 24, 2021
@Trott
Copy link
Member

Trott commented Aug 24, 2021

Closing this and we can retry it once #39858 lands.

@Trott Trott closed this Aug 24, 2021
nodejs-github-bot pushed a commit that referenced this pull request Aug 24, 2021
Refs: #39855 (comment)

PR-URL: #39858
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Zijian Liu <lxxyxzj@gmail.com>
targos pushed a commit that referenced this pull request Aug 24, 2021
Refs: #39855 (comment)

PR-URL: #39858
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Zijian Liu <lxxyxzj@gmail.com>
@Trott Trott deleted the actions/authors-update branch August 25, 2021 04:09
targos pushed a commit that referenced this pull request Sep 4, 2021
Refs: #39855 (comment)

PR-URL: #39858
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Zijian Liu <lxxyxzj@gmail.com>
@targos targos removed the commit-queue-failed An error occurred while landing this pull request using GitHub Actions. label Sep 5, 2021
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.

4 participants