-
Notifications
You must be signed in to change notification settings - Fork 30k
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: fix nits in README.md collaborators list #12866
Conversation
@imran-iq Does your e-mail remain the same? |
README.md
Outdated
@@ -278,7 +278,7 @@ more information about the governance of the Node.js project, see | |||
**Isaac Z. Schlueter** <i@izs.me> | |||
* [italoacasas](https://github.com/italoacasas) - | |||
**Italo A. Casas** <me@italoacasas.com> (he/him) | |||
* [iWuzHere](https://github.com/iWuzHere) - | |||
* [imran-iq](https://github.com/imran-iq) - |
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.
I think you might want to re-order these alphabetically :)
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.
Done)
I am not sure to land this without @Fishrock123, @Qard, @imran-iq showing up. |
I'm happy either way on casing of my name. 👍 |
I think this is probably fine to land. |
Landed in: dd06246 |
PR-URL: nodejs#12866 Fixes: nodejs#12865 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
Checklist
Affected core subsystem(s)
meta
Fixes: #12865
cc @Fishrock123, @Qard, @imran-iq