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

Closed
wants to merge 2 commits into from
Closed

meta: update AUTHORS #42844

wants to merge 2 commits into from

Conversation

nodejs-github-bot
Copy link
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.

@nodejs-github-bot nodejs-github-bot added the meta Issues and PRs related to the general management of the project. label Apr 24, 2022
AUTHORS Outdated
@@ -3453,5 +3453,7 @@ Daniel Roe <[email protected]>
Niyas Sait <[email protected]>
K.C.Ashish Kumar <[email protected]>
Niklas Mischkulnig <[email protected]>
[email protected] <npm team>
Copy link
Member

Choose a reason for hiding this comment

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

This looks like a (perhaps intentionally) malformed entry. I would think it should be npm team <[email protected]>. Whatever the case, it's a duplicate entry so we'll want to mailmap it to a single entry.

@Trott Trott closed this Apr 24, 2022
@Trott Trott deleted the actions/authors-update branch April 24, 2022 05:03
@Trott Trott restored the actions/authors-update branch April 24, 2022 05:03
@Trott Trott mentioned this pull request Apr 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

2 participants