Skip to content

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