-
Notifications
You must be signed in to change notification settings - Fork 30.8k
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 #19768
doc: update AUTHORS list #19768
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For some entries the corresponding name of the person can be found in their github profile. Should we always use that name?
AUTHORS
Outdated
Ujjwal Sharma <[email protected]> | ||
Wei-Wei Wu <[email protected]> | ||
punteek <[email protected]> | ||
Tobias Nießen <[email protected]> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This entry should be reflected in the mailmap.
AUTHORS
Outdated
Tobias Nießen <[email protected]> | ||
Ken Lin <[email protected]> | ||
Piotr Grzesik <[email protected]> | ||
feugy <[email protected]> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The name is "Damien Feugas" and should be reflected as such (he has a middle name but I guess that is not important?).
Updated with names found in github profiles |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@@ -3,6 +3,8 @@ Aaron Heckmann <[email protected]> <[email protected]> | |||
Aayush Ahuja <[email protected]> aayush.a <[email protected]> | |||
Abe Fettig <[email protected]> <[email protected]> | |||
Akito Ito <[email protected]> <[email protected]> | |||
Alejandro Estrada <[email protected]> | |||
Alejandro Estrada <[email protected]> <[email protected]> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Duplicate
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It has to be like that AFAICT. First entry for the name (it is lowercase in the commit log). Second entry for the email alias.
PR-URL: nodejs#19768 Reviewed-By: Ruben Bridgewater <[email protected]>
Landed in dca09a7 |
PR-URL: #19768 Reviewed-By: Ruben Bridgewater <[email protected]>
No description provided.