Skip to content

build: run AUTHORS update weekly - #40004

Closed
Trott wants to merge 2 commits into
nodejs:masterfrom
Trott:authors-cron
Closed

build: run AUTHORS update weekly#40004
Trott wants to merge 2 commits into
nodejs:masterfrom
Trott:authors-cron

Conversation

@Trott

@TrottTrott commented Sep 5, 2021

Copy link
Copy Markdown
Member

No description provided.

@nodejs-github-botnodejs-github-bot added the meta Issues and PRs related to the general management of the project. label Sep 5, 2021
targos
targos reviewed Sep 5, 2021
View reviewed changes
Comment thread.github/workflows/authors.yml Outdated
Mesteery
Mesteery reviewed Sep 5, 2021
View reviewed changes
Comment thread.github/workflows/authors.yml
@MesteeryMesteery mentioned this pull request Sep 5, 2021
Closed
@TrottTrott added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. commit-queue Add this label to land a pull request using GitHub Actions. labels Sep 7, 2021
@github-actionsgithub-actionsBot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Sep 7, 2021
@github-actions

github-actionsBot commented Sep 7, 2021

Copy link
Copy Markdown
Contributor

Landed in ff7a0df...773a989

nodejs-github-bot pushed a commit that referenced this pull request Sep 7, 2021
@Trott@nodejs-github-bot
build: run AUTHORS update weekly
773a989
PR-URL: #40004
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
BethGriggs pushed a commit that referenced this pull request Sep 21, 2021
@Trott@BethGriggs
build: run AUTHORS update weekly
0b30867
PR-URL: #40004
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
@BethGriggsBethGriggs mentioned this pull request Sep 21, 2021
Merged
1 task
@Trott
Trott deleted the authors-cron branch October 9, 2021 15:20
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jasnelljasnelljasnell approved these changes
@targostargostargos approved these changes
@aduh95aduh95aduh95 approved these changes
+1 more reviewer
@MesteeryMesteeryMesteery left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.metaIssues and PRs related to the general management of the project.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@Trott@jasnell@targos@aduh95@Mesteery@nodejs-github-bot