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: add CTC meeting minutes 2016-02-17 #5410

Closed
wants to merge 1 commit into from

Conversation

@bnoordhuis
Copy link
Member

LGTM. Condenses the promises discussion very well. :-)

@cjihrig
Copy link
Contributor

cjihrig commented Feb 24, 2016

LGTM

@mscdex mscdex added the doc Issues and PRs related to the documentations. label Feb 24, 2016
@jasnell
Copy link
Member

jasnell commented Mar 2, 2016

LGTM

@jasnell jasnell added the meta Issues and PRs related to the general management of the project. label Mar 2, 2016
jasnell pushed a commit that referenced this pull request Mar 2, 2016
PR-URL: #5410
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: James M Snell <jasnell@gmail.com>
@jasnell
Copy link
Member

jasnell commented Mar 2, 2016

@rvagg ... would it actually be possible to start putting these into the nodejs/tsc repository?

Fishrock123 pushed a commit that referenced this pull request Mar 2, 2016
PR-URL: #5410
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: James M Snell <jasnell@gmail.com>
@rvagg
Copy link
Member Author

rvagg commented Mar 9, 2016

To me it makes sense that they stay with "core" but perhaps that doesn't mean that they should go into the core repo, the TSC repo seems like the wrong place, maybe it's time to start a CTC repo for meta & core business stuff that doesn't stray far enough into TSC territory? We could use it for announcing and managing CTC meetings, for collecting and organising onboarding of new collaborators. Or is repo-proliferation a problem?

@williamkapke
Copy link
Contributor

If a CTC repo is created & these move there, I just ask that there is an announcement made here to let others know.

Side note: Whatever the Promises discussion was- the summary maybe should just say "Lengthly conversation: refer to audio". Maybe it wasn't lengthy? I'm not sure unless I listen to the audio. :(

@jasnell
Copy link
Member

jasnell commented Mar 9, 2016

Closing because this landed already.

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.

7 participants