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.

Sign up for GitHub

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

Jump to bottom

doc: Combined TSC #2085

Merged
merged 1 commit into from
Jul 16, 2015
Merged

doc: Combined TSC #2085

merged 1 commit into from
Jul 16, 2015

Conversation

Fishrock123
Copy link
Contributor

Updated, the old version can be found at #1710.

Everyone gave their consent in that thread, except for @mhdawson, who I've removed until I get a "+1" from.

R= @mikeal / @nodejs/tsc

Note: the list is currently in no particular order at all.

@Fishrock123 Fishrock123 added the doc Issues and PRs related to the documentations. label Jun 30, 2015
@Fishrock123 Fishrock123 mentioned this pull request Jun 30, 2015
Closed
@Fishrock123 Fishrock123 changed the title Combined TSC. doc: Combined TSC Jun 30, 2015
@mikeal
Copy link
Contributor

mikeal commented Jun 30, 2015

+1

@bnoordhuis
Copy link
Member

Commit log could be a little better though. Combined TSC what?

@rvagg
Copy link
Member

rvagg commented Jul 1, 2015

I'd like to see @mhdawson back in there, he's in the "official" list https://nodejs.org/about/organization/ and attends all the meetings

@mikeal
Copy link
Contributor

mikeal commented Jul 1, 2015

I'd like to see @mhdawson back in there, he's in the "official" list https://nodejs.org/about/organization/ and attends all the meetings

Agreed, that was probably an oversight from that day or so we were worried that we might be over the limit for IBM or something.

@Fishrock123 Fishrock123 force-pushed the combined-tsc branch 2 times, most recently from 01783f4 to 2dfb7a5 Compare July 7, 2015 16:55
@Fishrock123
Copy link
Contributor Author

Updated PTAL. It's changed quite a bit so I re-committed it under myself instead.

Ordered roughly in order of addition (since the collaborators still are). And added @mhdawson back, even though we've never actually gotten consent from him on that.

@bnoordhuis does this commit message look good?

@thefourtheye thefourtheye mentioned this pull request Jul 7, 2015
Closed
@Fishrock123 Fishrock123 self-assigned this Jul 16, 2015
@Fishrock123
doc: add TSC members from the combined project
4e92dbc 
These members were collaborators on the old joyent/node repo prior to
convergence of the Node.js and io.js projects under the Node
Foundation. This updates our list to represent that they are now also
part of the TSC (Technical Steering Committee), which was previously
known as the io.js TC.

PR-URL: nodejs#2085
Reviewed-By: Mikeal Rogers <mikeal.rogers@gmail.com>
Reviewed-By: Rod Vagg <rod@vagg.org>

Membership changes also previously reviewed by many more TSC & former
TC members in nodejs#1710
@Fishrock123 Fishrock123 merged commit 4e92dbc into nodejs:master Jul 16, 2015
@Fishrock123
Copy link
Contributor Author

Enough waiting on this, landed in 4e92dbc

@Fishrock123 Fishrock123 mentioned this pull request Jul 17, 2015
Merged
@orangemocha orangemocha mentioned this pull request Jul 29, 2015
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@Fishrock123 Fishrock123

Labels
doc Issues and PRs related to the documentations.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@Fishrock123 @mikeal @bnoordhuis @rvagg