doc: meeting notes from 2016-12-01 by joshgav · Pull Request #23 · nodejs/abi-stable-node · GitHub
Skip to content

doc: meeting notes from 2016-12-01#23

Merged
aruneshchandra merged 1 commit into
nodejs:api-prototype-6.2.0from
joshgav:collab-summit-meeting
Dec 7, 2016
Merged

doc: meeting notes from 2016-12-01#23
aruneshchandra merged 1 commit into
nodejs:api-prototype-6.2.0from
joshgav:collab-summit-meeting

Conversation

@joshgav

@joshgav joshgav commented Dec 2, 2016

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

doc

Description of change

notes from Collaboration Summit meeting

/cc @nodejs/api @nodejs/chakracore @nodejs/v8 @nodejs/spidernode @nodejs/ctc @nodejs/tsc

jasnell

This comment was marked as off-topic.

Sign in to view

aruneshchandra

This comment was marked as off-topic.

Sign in to view

aruneshchandra

This comment was marked as off-topic.

Sign in to view

jasnell

This comment was marked as off-topic.

Sign in to view

@joshgav

joshgav commented Dec 5, 2016 •
edited
Loading

Copy link
Copy Markdown
Author

s3ththompson

This comment was marked as off-topic.

Sign in to view

mhdawson

This comment was marked as off-topic.

Sign in to view

@fhinkel

fhinkel commented Dec 7, 2016

Copy link
Copy Markdown
Member

Sorry to be behind on this. We're reviewing the NAPI right now. Is it OK if we have results next week? /cc @nodejs/v8

@aruneshchandra

aruneshchandra commented Dec 7, 2016

Copy link
Copy Markdown
Contributor

@fhinkel yes its ok! BTW you are not behind :)

@joshgav
doc: meeting notes from 2016-12-01
040e077 
PR-URL: #23
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Arunesh Chandra <arunesh.chandra@microsoft.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
@joshgav

joshgav commented Dec 7, 2016

Copy link
Copy Markdown
Author

Updated, but I don't have push access in this repo. @mhdawson can you land? Thanks!

@aruneshchandra aruneshchandra merged commit f44a02c into nodejs:api-prototype-6.2.0 Dec 7, 2016
@aruneshchandra

aruneshchandra commented Dec 7, 2016

Copy link
Copy Markdown
Contributor

Landed it! Thanx @joshgav

@fhinkel

fhinkel commented Dec 12, 2016

Copy link
Copy Markdown
Member

jasongin pushed a commit to jasongin/abi-stable-node that referenced this pull request Jan 26, 2017
@jasongin
doc: meeting notes from 2016-12-01 (nodejs#23)
33dc071 
PR-URL: nodejs#23
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Arunesh Chandra <arunesh.chandra@microsoft.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jasnell jasnell jasnell approved these changes
@mhdawson mhdawson mhdawson approved these changes
+2 more reviewers
@s3ththompson s3ththompson s3ththompson left review comments
@aruneshchandra aruneshchandra aruneshchandra approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@joshgav @fhinkel @aruneshchandra @jasnell @s3ththompson @mhdawson