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: consolidate AUTHORS entries for jemjam #40328

Merged
merged 1 commit into from
Oct 7, 2021
Merged

Conversation

Trott
Copy link
Member

@Trott Trott commented Oct 5, 2021

I wasn't sure which of the two emails we have for @jemjam was the right one to list in the AUTHORS file, and then saw that there was a third email in their GitHub profile so went with that one.

@jemjam Let me know if you'd prefer to be listed in AUTHORS with a different email and I'll update .mailmap in this PR as appropriate.

@nodejs-github-bot nodejs-github-bot added the meta Issues and PRs related to the general management of the project. label Oct 5, 2021
@Trott Trott added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Oct 5, 2021
PR-URL: nodejs#40328
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Zijian Liu <lxxyxzj@gmail.com>
@Trott
Copy link
Member Author

Trott commented Oct 7, 2021

Landed in 09e8e14

@Trott Trott merged commit 09e8e14 into nodejs:master Oct 7, 2021
@Trott Trott deleted the mailmap-jb branch October 7, 2021 14:03
danielleadams pushed a commit to danielleadams/node that referenced this pull request Oct 8, 2021
PR-URL: nodejs#40328
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Zijian Liu <lxxyxzj@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
author ready PRs that have at least one approval, no pending requests for changes, and a CI started. 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