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: update minute-taking procedure for CTC #9425

Closed
wants to merge 1 commit into from

Conversation

Trott
Copy link
Member

@Trott Trott commented Nov 2, 2016

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

doc

Description of change

The doc says that the CTC moderator is responsible for taking minutes
and that the minutes for each agenda item should be a separate pull
request.

This change updates the doc to say that the moderator is responsible for
making sure that notes are taken, but not necessarily taking the notes
themselves. Furthermore, it removes the separate-pull-request
requirement as that is not what happens in practice.

@nodejs/ctc

@Trott Trott added doc Issues and PRs related to the documentations. ctc-review labels Nov 2, 2016
@nodejs-github-bot nodejs-github-bot added the meta Issues and PRs related to the general management of the project. label Nov 2, 2016
@joshgav
Copy link
Contributor

joshgav commented Nov 3, 2016

LGTM, it's hard to run a meeting and take notes at the same time.

I didn't interpret the original as requiring a separate PR for each agenda item, but no matter with this change.

@Trott
Copy link
Member Author

Trott commented Nov 4, 2016

I didn't interpret the original as requiring a separate PR

You're right. It probably was not intended that way.

@@ -142,8 +142,8 @@ items.
The CTC may invite persons or representatives from certain projects to
participate in a non-voting capacity.

The moderator is responsible for summarizing the discussion of each agenda item
and sending it as a pull request after the meeting.
The moderator is responsible for insuring that minutes are taken and that a pull
Copy link
Member

Choose a reason for hiding this comment

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

should insuring be ensuring instead ?

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 with nit

The doc says that the CTC moderator is responsible for taking minutes
and that the minutes for each agenda item should be a separate pull
request.

This change updates the doc to say that the moderator is responsible for
making sure that notes are taken, but not necessarily taking the notes
themselves. Furthermore, it removes the separate-pull-request
requirement as that is not what happens in practice.
Trott added a commit to Trott/io.js that referenced this pull request Nov 6, 2016
The doc says that the CTC moderator is responsible for taking minutes.

This change updates the doc to say that the moderator is responsible for
making sure that notes are taken, but not necessarily taking the notes
themselves.

PR-URL: nodejs#9425
Reviewed-By: Josh Gavant <josh.gavant@outlook.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
@Trott
Copy link
Member Author

Trott commented Nov 6, 2016

Landed in 7dd3fb8

@Trott Trott closed this Nov 6, 2016
evanlucas pushed a commit that referenced this pull request Nov 7, 2016
The doc says that the CTC moderator is responsible for taking minutes.

This change updates the doc to say that the moderator is responsible for
making sure that notes are taken, but not necessarily taking the notes
themselves.

PR-URL: #9425
Reviewed-By: Josh Gavant <josh.gavant@outlook.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
@Trott Trott removed the ctc-review label Nov 16, 2016
MylesBorins pushed a commit that referenced this pull request Nov 22, 2016
The doc says that the CTC moderator is responsible for taking minutes.

This change updates the doc to say that the moderator is responsible for
making sure that notes are taken, but not necessarily taking the notes
themselves.

PR-URL: #9425
Reviewed-By: Josh Gavant <josh.gavant@outlook.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
MylesBorins pushed a commit that referenced this pull request Nov 22, 2016
The doc says that the CTC moderator is responsible for taking minutes.

This change updates the doc to say that the moderator is responsible for
making sure that notes are taken, but not necessarily taking the notes
themselves.

PR-URL: #9425
Reviewed-By: Josh Gavant <josh.gavant@outlook.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
This was referenced Nov 22, 2016
@Trott Trott deleted the minutes branch January 13, 2022 22:44
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. meta Issues and PRs related to the general management of the project.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants