Skip to content

doc: update CTC governance information#7719

Closed
Trott wants to merge 3 commits intonodejs:masterfrom
Trott:jointly
Closed

doc: update CTC governance information#7719
Trott wants to merge 3 commits intonodejs:masterfrom
Trott:jointly

Conversation

@Trott
Copy link
Member

@Trott Trott commented Jul 13, 2016

Checklist
  • documentation is changed or added
  • commit message follows commit guidelines
Affected core subsystem(s)

meta

Description of change

Update some outdated material. Provide some minor fixes. Wrap to 80
characters.

@nodejs/ctc

@Trott Trott added the meta Issues and PRs related to the general management of the project. label Jul 13, 2016
vsemozhetbyt
vsemozhetbyt reviewed Jul 13, 2016
View reviewed changes
GOVERNANCE.md Outdated
Copy link
Contributor

@vsemozhetbyt vsemozhetbyt Jul 13, 2016

Choose a reason for hiding this comment

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

should be streamed?

Copy link
Member Author

@Trott Trott Jul 13, 2016

Choose a reason for hiding this comment

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

Ah, yes thanks, will fix!

@rvagg
Copy link
Member

rvagg commented Jul 14, 2016

aye, lgtm

@bnoordhuis
Copy link
Member

bnoordhuis commented Jul 14, 2016

LGTM

Fishrock123
Fishrock123 reviewed Jul 14, 2016
View reviewed changes
GOVERNANCE.md Outdated
Copy link
Contributor

@Fishrock123 Fishrock123 Jul 14, 2016

Choose a reason for hiding this comment

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

in a voice conference call

Copy link
Member Author

@Trott Trott Jul 14, 2016 •
edited
Loading

Choose a reason for hiding this comment

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

Good point; I'll add that.

@Fishrock123
Copy link
Contributor

Fishrock123 commented Jul 14, 2016

lgtm minus nits

Trott added 3 commits July 14, 2016 13:41
@Trott
doc: update CTC governance information
893d685 
Update some outdated material. Provide some minor fixes. Wrap to 80
characters.
@Trott
squash: fix typo
3e9b94f
@Trott
squash: in a conference call
e46d5e1
@mhdawson
Copy link
Member

mhdawson commented Jul 14, 2016

LGTM

@Trott
Copy link
Member Author

Trott commented Jul 15, 2016

@nodejs/ctc Would be great to get some more LGTMs on this from CTC folks so I can land it in good conscience rather than putting it on the agenda for the next meeting. I imagine the other proposed changes might still need to go to the CTC meeting, but these seem mostly uncontroversial...

@misterdjules
Copy link

misterdjules commented Jul 15, 2016

LGTM

3 similar comments
@cjihrig
Copy link
Contributor

cjihrig commented Jul 17, 2016

LGTM

@ofrobots
Copy link
Contributor

ofrobots commented Jul 17, 2016

LGTM

@evanlucas
Copy link
Contributor

evanlucas commented Jul 18, 2016

LGTM

@indutny
Copy link
Member

indutny commented Jul 18, 2016 •
edited
Loading

LGTM/ЛГТМ

@ChALkeR
Copy link
Member

ChALkeR commented Jul 18, 2016

LGTM

1 similar comment
@trevnorris
Copy link
Contributor

trevnorris commented Jul 18, 2016

LGTM

@Trott
Copy link
Member Author

Trott commented Jul 19, 2016

11 LGTMs plus me = 12 CTC folks in favor (out of 18), none opposed...

... Landed in 4ae24e1

@Trott Trott closed this Jul 19, 2016
Trott added a commit that referenced this pull request Jul 19, 2016
@Trott
doc: update CTC governance information
4ae24e1 
Update some outdated material. Provide some minor fixes. Wrap to 80
characters.

PR-URL: #7719
Reviewed-By: Rod Vagg <rod@vagg.org>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Julien Gilli <jgilli@nodejs.org>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ali Ijaz Sheikh <ofrobots@google.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Fedor Indutny <fedor.indutny@gmail.com>
Reviewed-By: Сковорода Никита Андреевич <chalkerx@gmail.com>
Reviewed-By: Trevor Norris <trev.norris@gmail.com>
evanlucas pushed a commit that referenced this pull request Jul 19, 2016
@Trott @evanlucas
doc: update CTC governance information
059a721 
Update some outdated material. Provide some minor fixes. Wrap to 80
characters.

PR-URL: #7719
Reviewed-By: Rod Vagg <rod@vagg.org>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Julien Gilli <jgilli@nodejs.org>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ali Ijaz Sheikh <ofrobots@google.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Fedor Indutny <fedor.indutny@gmail.com>
Reviewed-By: Сковорода Никита Андреевич <chalkerx@gmail.com>
Reviewed-By: Trevor Norris <trev.norris@gmail.com>
@evanlucas evanlucas mentioned this pull request Jul 19, 2016
Merged
evanlucas pushed a commit that referenced this pull request Jul 20, 2016
@Trott @evanlucas
doc: update CTC governance information
59ed303 
Update some outdated material. Provide some minor fixes. Wrap to 80
characters.

PR-URL: #7719
Reviewed-By: Rod Vagg <rod@vagg.org>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Julien Gilli <jgilli@nodejs.org>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ali Ijaz Sheikh <ofrobots@google.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Fedor Indutny <fedor.indutny@gmail.com>
Reviewed-By: Сковорода Никита Андреевич <chalkerx@gmail.com>
Reviewed-By: Trevor Norris <trev.norris@gmail.com>
MylesBorins pushed a commit that referenced this pull request Sep 9, 2016
@Trott
doc: update CTC governance information
2fda453 
Update some outdated material. Provide some minor fixes. Wrap to 80
characters.

PR-URL: #7719
Reviewed-By: Rod Vagg <rod@vagg.org>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Julien Gilli <jgilli@nodejs.org>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ali Ijaz Sheikh <ofrobots@google.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Fedor Indutny <fedor.indutny@gmail.com>
Reviewed-By: Сковорода Никита Андреевич <chalkerx@gmail.com>
Reviewed-By: Trevor Norris <trev.norris@gmail.com>
MylesBorins pushed a commit that referenced this pull request Sep 28, 2016
@Trott
doc: update CTC governance information
304e7ae 
Update some outdated material. Provide some minor fixes. Wrap to 80
characters.

PR-URL: #7719
Reviewed-By: Rod Vagg <rod@vagg.org>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Julien Gilli <jgilli@nodejs.org>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ali Ijaz Sheikh <ofrobots@google.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Fedor Indutny <fedor.indutny@gmail.com>
Reviewed-By: Сковорода Никита Андреевич <chalkerx@gmail.com>
Reviewed-By: Trevor Norris <trev.norris@gmail.com>
rvagg pushed a commit that referenced this pull request Oct 18, 2016
@Trott @rvagg
doc: update CTC governance information
3ac6a26 
Update some outdated material. Provide some minor fixes. Wrap to 80
characters.

PR-URL: #7719
Reviewed-By: Rod Vagg <rod@vagg.org>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Julien Gilli <jgilli@nodejs.org>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ali Ijaz Sheikh <ofrobots@google.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Fedor Indutny <fedor.indutny@gmail.com>
Reviewed-By: Сковорода Никита Андреевич <chalkerx@gmail.com>
Reviewed-By: Trevor Norris <trev.norris@gmail.com>
MylesBorins pushed a commit that referenced this pull request Oct 26, 2016
@Trott @MylesBorins
doc: update CTC governance information
590c52a 
Update some outdated material. Provide some minor fixes. Wrap to 80
characters.

PR-URL: #7719
Reviewed-By: Rod Vagg <rod@vagg.org>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Julien Gilli <jgilli@nodejs.org>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ali Ijaz Sheikh <ofrobots@google.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Fedor Indutny <fedor.indutny@gmail.com>
Reviewed-By: Сковорода Никита Андреевич <chalkerx@gmail.com>
Reviewed-By: Trevor Norris <trev.norris@gmail.com>
@MylesBorins MylesBorins mentioned this pull request Oct 26, 2016
Closed
@Trott Trott deleted the jointly branch January 13, 2022 22:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

meta Issues and PRs related to the general management of the project.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.