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: align README/.mailmap/AUTHORS email entries #39505

Closed
wants to merge 1 commit into from

Conversation

Trott
Copy link
Member

@Trott Trott commented Jul 24, 2021

The email address for Weijia Wang in the AUTHORS and .mailmap files is
different from the email address in the README.md file. Based on their
GitHub profile and commit metadata, the one in the README.md file is the
preferred email. Updating .mailmap and AUTHORS to reflect it.

The email address for Weijia Wang in the AUTHORS and .mailmap files is
different from the email address in the README.md file. Based on their
GitHub profile and commit metadata, the one in the README.md file is the
preferred email. Updating .mailmap and AUTHORS to reflect it.
@Trott Trott requested a review from starkwang July 24, 2021 12:39
@nodejs-github-bot nodejs-github-bot added the meta Issues and PRs related to the general management of the project. label Jul 24, 2021
@Trott
Copy link
Member Author

Trott commented Jul 24, 2021

@starkwang If you prefer a different email, please comment. Thanks!

(I'm trying to align the README.md emails with .mailmap for tooling.)

Copy link
Contributor

@starkwang starkwang left a comment

Choose a reason for hiding this comment

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

@Trott Thanks for your fix!

@Trott Trott added the commit-queue Add this label to land a pull request using GitHub Actions. label Jul 31, 2021
@github-actions github-actions bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Jul 31, 2021
@github-actions
Copy link
Contributor

Landed in 8be3b91...1a1793e

@github-actions github-actions bot closed this Jul 31, 2021
nodejs-github-bot pushed a commit that referenced this pull request Jul 31, 2021
The email address for Weijia Wang in the AUTHORS and .mailmap files is
different from the email address in the README.md file. Based on their
GitHub profile and commit metadata, the one in the README.md file is the
preferred email. Updating .mailmap and AUTHORS to reflect it.

PR-URL: #39505
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Weijia Wang <starkwang@126.com>
danielleadams pushed a commit that referenced this pull request Aug 16, 2021
The email address for Weijia Wang in the AUTHORS and .mailmap files is
different from the email address in the README.md file. Based on their
GitHub profile and commit metadata, the one in the README.md file is the
preferred email. Updating .mailmap and AUTHORS to reflect it.

PR-URL: #39505
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Weijia Wang <starkwang@126.com>
targos pushed a commit that referenced this pull request Sep 4, 2021
The email address for Weijia Wang in the AUTHORS and .mailmap files is
different from the email address in the README.md file. Based on their
GitHub profile and commit metadata, the one in the README.md file is the
preferred email. Updating .mailmap and AUTHORS to reflect it.

PR-URL: #39505
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Weijia Wang <starkwang@126.com>
@Trott Trott deleted the ww-email branch September 25, 2022 17:09
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.

4 participants