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

doc: mention cc-ing nodejs/python team for reviews #10637

Closed
wants to merge 1 commit into from

Conversation

addaleax
Copy link
Member

@addaleax addaleax commented Jan 5, 2017

Add the @nodejs/python github team to the table of people to /cc for reviews on python code.

I was talking about this a while ago with @thefourtheye, and we did create the team, but I think we have (I have? 😄) just forgotten to add the team to the table in doc/onboarding-extras.md.

Add the nodejs/python github team to the table of people to /cc for
reviews on python code.
@nodejs-github-bot nodejs-github-bot added the doc Issues and PRs related to the documentations. label Jan 5, 2017
@thefourtheye
Copy link
Contributor

Oopsie. I totally forgot. Thanks @addaleax :-)

Copy link
Member

@mhdawson mhdawson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jasnell
Copy link
Member

jasnell commented Jan 5, 2017

This should not need to wait the full 48 hours to land

Copy link
Member

@gibfahn gibfahn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, did not know this was a thing!

@addaleax
Copy link
Member Author

addaleax commented Jan 6, 2017

Alright, went ahead and landed this in 103f6d7

@addaleax addaleax closed this Jan 6, 2017
@addaleax addaleax deleted the doc-cc-nodejs-python branch January 6, 2017 01:02
addaleax added a commit that referenced this pull request Jan 6, 2017
Add the nodejs/python github team to the table of people to /cc for
reviews on python code.

PR-URL: #10637
Reviewed-By: Jeremiah Senkpiel <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Sakthipriyan Vairamani <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Gibson Fahnestock <[email protected]>
italoacasas pushed a commit to italoacasas/node that referenced this pull request Jan 18, 2017
Add the nodejs/python github team to the table of people to /cc for
reviews on python code.

PR-URL: nodejs#10637
Reviewed-By: Jeremiah Senkpiel <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Sakthipriyan Vairamani <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Gibson Fahnestock <[email protected]>
italoacasas pushed a commit to italoacasas/node that referenced this pull request Jan 19, 2017
Add the nodejs/python github team to the table of people to /cc for
reviews on python code.

PR-URL: nodejs#10637
Reviewed-By: Jeremiah Senkpiel <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Sakthipriyan Vairamani <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Gibson Fahnestock <[email protected]>
italoacasas pushed a commit to italoacasas/node that referenced this pull request Jan 24, 2017
Add the nodejs/python github team to the table of people to /cc for
reviews on python code.

PR-URL: nodejs#10637
Reviewed-By: Jeremiah Senkpiel <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Sakthipriyan Vairamani <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Gibson Fahnestock <[email protected]>
italoacasas pushed a commit to italoacasas/node that referenced this pull request Jan 27, 2017
Add the nodejs/python github team to the table of people to /cc for
reviews on python code.

PR-URL: nodejs#10637
Reviewed-By: Jeremiah Senkpiel <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Sakthipriyan Vairamani <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Gibson Fahnestock <[email protected]>
@italoacasas italoacasas mentioned this pull request Jan 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc Issues and PRs related to the documentations.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants