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: revise .mailmap for README consistency #39457

Merged
merged 1 commit into from
Jul 24, 2021
Merged

meta: revise .mailmap for README consistency #39457

merged 1 commit into from
Jul 24, 2021

Conversation

Trott
Copy link
Member

@Trott Trott commented Jul 19, 2021

Update entry in .mailmap to reflect the same entry in README for an
existing collaborator.

@Trott Trott requested a review from yorkie July 19, 2021 22:31
@nodejs-github-bot nodejs-github-bot added the meta Issues and PRs related to the general management of the project. label Jul 19, 2021
@Trott
Copy link
Member Author

Trott commented Jul 19, 2021

@yorkie If you'd prefer to have the README changed, instead of .mailmap and AUTHORS, we can do that instead.

@Trott Trott added commit-queue Add this label to land a pull request using GitHub Actions. and removed commit-queue Add this label to land a pull request using GitHub Actions. labels Jul 24, 2021
Update entry in .mailmap to reflect the same entry in README for an
existing collaborator.

PR-URL: nodejs#39457
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
@Trott Trott merged commit 6140b37 into nodejs:master Jul 24, 2021
@Trott
Copy link
Member Author

Trott commented Jul 24, 2021

Landed in 6140b37

@Trott Trott deleted the ylf branch July 24, 2021 05:08
targos pushed a commit that referenced this pull request Jul 25, 2021
Update entry in .mailmap to reflect the same entry in README for an
existing collaborator.

PR-URL: #39457
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
@BethGriggs BethGriggs mentioned this pull request Jul 26, 2021
richardlau pushed a commit that referenced this pull request Jul 29, 2021
Update entry in .mailmap to reflect the same entry in README for an
existing collaborator.

PR-URL: #39457
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
BethGriggs pushed a commit that referenced this pull request Jul 29, 2021
Update entry in .mailmap to reflect the same entry in README for an
existing collaborator.

PR-URL: #39457
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
targos pushed a commit that referenced this pull request Sep 4, 2021
Update entry in .mailmap to reflect the same entry in README for an
existing collaborator.

PR-URL: #39457
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
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