Skip to content

meta: update AUTHORS - #40392

Merged
Trott merged 1 commit into
masterfrom
actions/authors-update
Oct 10, 2021
Merged

meta: update AUTHORS#40392
Trott merged 1 commit into
masterfrom
actions/authors-update

Conversation

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Here are some new additions to the AUTHORS file. This is an automatically generated PR by the authors.yml GitHub Action, which runs tools/update-authors.js.

{{ env.DUPLICATES }}

@nodejs-github-botnodejs-github-bot added the meta Issues and PRs related to the general management of the project. label Oct 10, 2021
@Trott

Copy link
Copy Markdown
Member

{{ env.DUPLICATES }}

Well, that didn't work.

@TrottTrott 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.

I manually confirmed that none of these are duplicates so this is ready to land.

@TrottTrott added fast-track PRs that do not need to wait for 48 hours to land. author ready PRs that have at least one approval, no pending requests for changes, and a CI started. labels Oct 10, 2021
@github-actions

Copy link
Copy Markdown
Contributor

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

@aduh95aduh95 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

RSLGTM

@aduh95aduh95 added the commit-queue Add this label to land a pull request using GitHub Actions. label Oct 10, 2021
@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 Oct 10, 2021
@github-actions

Copy link
Copy Markdown
Contributor
Commit Queue failed
- Loading data for nodejs/node/pull/40392
✔ Done loading data for nodejs/node/pull/40392
----------------------------------- PR info ------------------------------------
Title meta: update AUTHORS (#40392)
⚠ Could not retrieve the email or name of the PR author's from user's GitHub profile!
Branch nodejs-github-bot:actions/authors-update -> nodejs:master
Labels meta, fast-track, author ready
Commits 1
- meta: update AUTHORS
Committers 1
- Node.js GitHub Bot PR-URL: https://github.com/nodejs/node/pull/40392
Reviewed-By: Rich Trott Reviewed-By: Tobias Nießen Reviewed-By: Antoine du Hamel ------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/40392
Reviewed-By: Rich Trott Reviewed-By: Tobias Nießen Reviewed-By: Antoine du Hamel --------------------------------------------------------------------------------
ℹ This PR was created on Sun, 10 Oct 2021 00:14:07 GMT
✔ Approvals: 3
✔ - Rich Trott (@Trott) (TSC): https://github.com/nodejs/node/pull/40392#pullrequestreview-775618121
✔ - Tobias Nießen (@tniessen) (TSC): https://github.com/nodejs/node/pull/40392#pullrequestreview-775620278
✔ - Antoine du Hamel (@aduh95) (TSC): https://github.com/nodejs/node/pull/40392#pullrequestreview-775713003
ℹ 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/1326708013

PR-URL: #40392
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
@Trott
Trottforce-pushed the actions/authors-update branch from 31b9603 to 49b7ec9CompareOctober 10, 2021 22:28
@Trott

Copy link
Copy Markdown
Member

Landed in 49b7ec9

@Trott
Trott merged commit 49b7ec9 into masterOct 10, 2021
@Trott
Trott deleted the actions/authors-update branch October 10, 2021 22:29
targos pushed a commit that referenced this pull request Oct 13, 2021
PR-URL: #40392
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
@BethGriggsBethGriggs mentioned this pull request Oct 14, 2021
2 tasks
@richardlaurichardlau mentioned this pull request Oct 18, 2021
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.commit-queue-failedAn error occurred while landing this pull request using GitHub Actions.fast-trackPRs that do not need to wait for 48 hours to land.metaIssues 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

@nodejs-github-bot@Trott@tniessen@aduh95