-
Notifications
You must be signed in to change notification settings - Fork 30.1k
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: update AUTHORS #42760
meta: update AUTHORS #42760
Conversation
By the way, if someone is feeling like they want to improve this, the script detects probable duplicates. (Sometimes it flags things as duplicates that aren't--you have to go and look to see if it's two different GitHub accounts that happen to be people with the same name or if it's the same person using two different email addresses. Usually, it's the latter, but not always.) However, the script/GitHub Action doesn't put that information in the pull request. I always have to go and look for it. (For this one, go to https://github.com/nodejs/node/runs/6050766933?check_suite_focus=true, and expand It would be great if that information ended up in the pull request. Sure, it still takes a human to investigate, but at least the information isn't hidden. |
2d99a60
to
f41f891
Compare
Fast-track has been requested by @Trott. Please 👍 to approve. |
Landed in 9724ca2 |
PR-URL: nodejs#42760 Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Darshan Sen <raisinten@gmail.com>
PR-URL: #42760 Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Darshan Sen <raisinten@gmail.com>
PR-URL: #42760 Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Darshan Sen <raisinten@gmail.com>
PR-URL: #42760 Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Darshan Sen <raisinten@gmail.com>
PR-URL: #42760 Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Darshan Sen <raisinten@gmail.com>
PR-URL: #42760 Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Darshan Sen <raisinten@gmail.com>
PR-URL: nodejs/node#42760 Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Darshan Sen <raisinten@gmail.com>
Here are some new additions to the AUTHORS file. This is an automatically generated PR by the
authors.yml
GitHub Action, which runstools/update-authors.js
.