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 @shisama as a collaborator #23850

Closed
addaleax opened this issue Oct 24, 2018 · 7 comments
Closed

Nominating @shisama as a collaborator #23850

addaleax opened this issue Oct 24, 2018 · 7 comments
Labels
meta Issues and PRs related to the general management of the project.

Comments

@addaleax
Copy link
Member

I'd like to nominate @shisama as a Collaborator. With 30 commits and consistent engagement in Node.js core, this should probably have happened a while ago :)

Commits: https://github.com/nodejs/node/commits?author=shisama
Comments: https://github.com/nodejs/node/search?q=commenter%3Ashisama&type=Issues

@addaleax addaleax added the meta Issues and PRs related to the general management of the project. label Oct 24, 2018
@gireeshpunathil
Copy link
Member

+1

4 similar comments
@mhdawson
Copy link
Member

+1

@cjihrig
Copy link
Contributor

cjihrig commented Oct 24, 2018

+1

@BridgeAR
Copy link
Member

+1

@hiroppy
Copy link
Member

hiroppy commented Oct 25, 2018

+1

@fhinkel
Copy link
Member

fhinkel commented Oct 30, 2018

+1

@joyeecheung joyeecheung added the tsc-agenda Issues and PRs to discuss during the meetings of the TSC. label Oct 31, 2018
@joyeecheung
Copy link
Member

This has been open for a week so according to the GOVERNANCE.md the nomination has now been considered accepted. Marking it as tsc-agenda so that it gets announced at the TSC meeting.

@Trott Trott removed the tsc-agenda Issues and PRs to discuss during the meetings of the TSC. label Nov 5, 2018
@shisama shisama closed this as completed in a49b672 Nov 6, 2018
targos pushed a commit that referenced this issue Nov 6, 2018
PR-URL: #24136
Fixes: #23850
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Vse Mozhet Byt <[email protected]>
kiyomizumia pushed a commit to kiyomizumia/node that referenced this issue Nov 15, 2018
PR-URL: nodejs#24136
Fixes: nodejs#23850
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Vse Mozhet Byt <[email protected]>
codebytere pushed a commit that referenced this issue Nov 29, 2018
PR-URL: #24136
Fixes: #23850
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Vse Mozhet Byt <[email protected]>
MylesBorins pushed a commit that referenced this issue Nov 29, 2018
PR-URL: #24136
Fixes: #23850
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Vse Mozhet Byt <[email protected]>
MylesBorins pushed a commit that referenced this issue Dec 3, 2018
PR-URL: #24136
Fixes: #23850
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Vse Mozhet Byt <[email protected]>
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 a pull request may close this issue.

9 participants