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-01-20 #4904

Closed
wants to merge 1 commit into from

Conversation

@JungMinu JungMinu added the doc Issues and PRs related to the documentations. label Jan 27, 2016

Alexis: MS submitted a PR for supporting Chakracore, change in node is fairly small — the shim is in the deps folder. A few CTC members have expressed interest in landing. I was wondering if anyone had opinions about how to proceed with this?

Domenic: I’m curious about what level of support is implied. Landing it reduces the requirement of maintaining a separate fork, but
Copy link
Member

Choose a reason for hiding this comment

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

Cut short?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yeah, there are a few in this one unfortunately. I think the pace of discussion was a bit too fast in this meeting for good minuting. We had a lot to cover and were all conscious of time, plus there were some pretty strong opinions to be had!

@jasnell
Copy link
Member

jasnell commented Jan 27, 2016

LGTM

@jasnell jasnell added the meta Issues and PRs related to the general management of the project. label Jan 27, 2016
@bnoordhuis
Copy link
Member

LGTM. Too bad about the gaps but c'est la vie.

silverwind pushed a commit that referenced this pull request Feb 4, 2016
PR-URL: #4904
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Roman Reiss <[email protected]>
@silverwind
Copy link
Contributor

Landed in e31bda8.

@silverwind silverwind closed this Feb 4, 2016
rvagg added a commit that referenced this pull request Feb 8, 2016
PR-URL: #4904
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Roman Reiss <[email protected]>
scovetta pushed a commit to scovetta/node that referenced this pull request Apr 2, 2016
PR-URL: nodejs#4904
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Roman Reiss <[email protected]>
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