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

Nominating Akhil Marsonya (marsonya) to be a collaborator #41781

Closed
gireeshpunathil opened this issue Jan 31, 2022 · 8 comments · Fixed by #41991
Closed

Nominating Akhil Marsonya (marsonya) to be a collaborator #41781

gireeshpunathil opened this issue Jan 31, 2022 · 8 comments · Fixed by #41991

Comments

@gireeshpunathil
Copy link
Member

What is the problem this feature will solve?

Lack of marsonya in collaborator team

What is the feature you are proposing to solve the problem?

marsonya in collaborator team

What alternatives have you considered?

marsonya continue to be a triager - which potentially ignores the acknowledgement they deserve.

I would like to nominate Akhil Marsonya (marsonya) to be a collaborator. Akhil is a member of the triage team, and has made effective use of the role to make inroads into the codebase as well as helping Node.js users. Below is the summary of their contributions:

Pull requests and issues opened throughout the Node.js organization: https://github.com/search?q=author:marsonya+org:nodejs
Commits in nodejs/node: https://github.com/nodejs/node/commits?author=marsonya
Pull requests and issues in nodejs/node: https://github.com/nodejs/node/issues?q=author:marsonya
Comments on pull requests and issues in nodejs/node: https://github.com/nodejs/node/issues?q=commenter:marsonya
Reviews on pull requests in nodejs/node: https://github.com/nodejs/node/pulls?q=reviewed-by:marsonya
Help provided to end-users and novice contributors: https://github.com/nodejs/help/issues?q=commenter:marsonya

@gireeshpunathil gireeshpunathil added feature request Issues that request new features to be added to Node.js. and removed feature request Issues that request new features to be added to Node.js. labels Jan 31, 2022
@marsonya
Copy link
Member

Thank you!! It's an Honour 😊

@aduh95
Copy link
Contributor

aduh95 commented Jan 31, 2022

What is the problem this feature will solve?

Lack of marsonya in collaborator team

I can reproduce on master and all the active release lines, we should definitely fix this!

Jokes aside, it's a warm +1 from me 🤗

@mhdawson
Copy link
Member

+1 from me

@benjamingr
Copy link
Member

Hey, not a block but I would like to see demonstrated competence in the codebase to be able to trust them with landing stuff. Stuff like #41815 is a bit worrying to me to be completely honest <3

@Trott
Copy link
Member

Trott commented Feb 4, 2022

Hey, not a block but I would like to see demonstrated competence in the codebase to be able to trust them with landing stuff. Stuff like #41815 is a bit worrying to me to be completely honest <3

I'm OK with people freely admitting when they don't know something and asking. I prefer that to people not asking or pretending they know more than they do. Concerns for me would be signs that someone is not aware of the limits of their knowledge and/or does not act with appropriate caution on things they don't fully understand. (I'm not seeing those problems here.)

@benjamingr
Copy link
Member

To be clear the issue I have isn't with not-knowing (I don't know most of the code). It's about collaboratorship implying someone's self-sufficiency in the codebase and the capability to look these things up.

Looking at his many helpful commits, the help and the other interactions - I see a tremendous amount of helpfulness and willingness to help the project. What I'm missing is demonstrated competence in the Node.js core codebase itself.

That said - I acknowledge the meaning of collaborator may have shifted a bit or that other people might have a different impression and I haven't had a single negative interaction (and did have several good ones) with Akhil. So to be clear I am not blocking this nomination and I believe that if there are multiple 👍 s from the TSC it's totally fine for it to go through.

@benjamingr
Copy link
Member

for what it's worth after this comment I am satisfied with @marsonya 's ability in the code. Which again I wasn't blocking on in the first place so mostly theoretical stuff :)

@gireeshpunathil
Copy link
Member Author

this nomination has passed; I will connect with Akhil to schedule the onboarding.

nodejs-github-bot pushed a commit that referenced this issue Feb 15, 2022
Fixes: #41781

PR-URL: #41991
Reviewed-By: Mestery <mestery@protonmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
bengl pushed a commit to bengl/node that referenced this issue Feb 21, 2022
Fixes: nodejs#41781

PR-URL: nodejs#41991
Reviewed-By: Mestery <mestery@protonmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
bengl pushed a commit to bengl/node that referenced this issue Feb 21, 2022
Fixes: nodejs#41781

PR-URL: nodejs#41991
Reviewed-By: Mestery <mestery@protonmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
bengl pushed a commit that referenced this issue Feb 21, 2022
Fixes: #41781

PR-URL: #41991
Reviewed-By: Mestery <mestery@protonmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
bengl pushed a commit that referenced this issue Feb 22, 2022
Fixes: #41781

PR-URL: #41991
Reviewed-By: Mestery <mestery@protonmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
danielleadams pushed a commit that referenced this issue Apr 21, 2022
Fixes: #41781

PR-URL: #41991
Reviewed-By: Mestery <mestery@protonmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
danielleadams pushed a commit that referenced this issue Apr 24, 2022
Fixes: #41781

PR-URL: #41991
Reviewed-By: Mestery <mestery@protonmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants