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

doc: update AUTHORS list #29608

Closed
wants to merge 4 commits into from
Closed

Conversation

addaleax
Copy link
Member

Update the authors list, and make the .mailmap processing case-insensitive so that @MylesBorins doesn’t end up with multiple entries due to different casing in the email 🙂

This is to accommodate Myles Borins otherwise ending up with
multiple entries due to different casing in the email 🙂
@nodejs-github-bot nodejs-github-bot added meta Issues and PRs related to the general management of the project. tools Issues and PRs related to the tools directory. labels Sep 18, 2019
@@ -44,8 +45,10 @@ Ben Noordhuis <info@bnoordhuis.nl> <bnoordhuis@bender.(none)>
Ben Noordhuis <info@bnoordhuis.nl> <ben@strongloop.com>
Ben Taber <ben.taber@gmail.com>
Benjamin Coe <bencoe@gmail.com> <ben@npmjs.com>
Benjamin Coe <bencoe@gmail.com> <bencoe@google.com>
Copy link
Member Author

Choose a reason for hiding this comment

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

@bcoe Is the gmail one your preferred one? That’s what this is assuming here but please correct me if I’m wrong.

.mailmap Show resolved Hide resolved
.mailmap Show resolved Hide resolved
AUTHORS Outdated Show resolved Hide resolved
AUTHORS Outdated Show resolved Hide resolved
.mailmap Show resolved Hide resolved
@Trott
Copy link
Member

Trott commented Sep 24, 2019

@Trott Trott added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Sep 24, 2019
@Trott
Copy link
Member

Trott commented Sep 25, 2019

Landed in 7899a96...81fee01

@Trott Trott closed this Sep 25, 2019
Trott pushed a commit that referenced this pull request Sep 25, 2019
This is to accommodate Myles Borins otherwise ending up with
multiple entries due to different casing in the email 🙂

PR-URL: #29608
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com>
Reviewed-By: Saúl Ibarra Corretgé <saghul@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Trott pushed a commit that referenced this pull request Sep 25, 2019
PR-URL: #29608
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com>
Reviewed-By: Saúl Ibarra Corretgé <saghul@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
targos pushed a commit that referenced this pull request Oct 1, 2019
This is to accommodate Myles Borins otherwise ending up with
multiple entries due to different casing in the email 🙂

PR-URL: #29608
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com>
Reviewed-By: Saúl Ibarra Corretgé <saghul@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
targos pushed a commit that referenced this pull request Oct 1, 2019
PR-URL: #29608
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com>
Reviewed-By: Saúl Ibarra Corretgé <saghul@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@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. tools Issues and PRs related to the tools directory.
Projects
None yet
Development

Successfully merging this pull request may close these issues.