FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

meta: update AUTHORS by github-actions[bot] · Pull Request #39855 · nodejs/node · GitHub

/ node Public

meta: update AUTHORS - #39855

Closed
github-actions[bot] wants to merge 1 commit into
masterfrom
actions/authors-update
Closed

meta: update AUTHORS#39855
github-actions[bot] wants to merge 1 commit into
masterfrom
actions/authors-update

Conversation

Copy link
Copy Markdown
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 Bot added the meta Issues and PRs related to the general management of the project. label Aug 24, 2021

Trott left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

I checked that these aren't duplicates. I think this can land.

Trott added the fast-track PRs that do not need to wait for 48 hours to land. label Aug 24, 2021

Copy link
Copy Markdown
Contributor Author

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

bnb commented Aug 24, 2021

Copy link
Copy Markdown
Contributor

yay! 🎉

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

Copy link
Copy Markdown
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 commented Aug 24, 2021

Copy link
Copy Markdown
Member

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

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 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 removed the commit-queue-failed An error occurred while landing this pull request using GitHub Actions. label Sep 5, 2021
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

5 participants


Back | FazBrowse Home | New Git URL