Skip to content
This repository was archived by the owner on Feb 21, 2018. It is now read-only.

Add minutes for meeting 21 Apr 2016 - #19

Closed
mhdawson wants to merge 4 commits into
nodejs:masterfrom
mhdawson:master
Closed

mhdawson wants to merge 4 commits into
nodejs:masterfrom
mhdawson:master

Conversation

@mhdawson

Copy link
Copy Markdown
Member

@nodejs/api please review/comment

@mhdawson mhdawson mentioned this pull request Apr 21, 2016
Comment thread wg-meetings/2016-04-21.md Outdated
This doc has some initial notes/info about investigation
by Stefan/Ian so far:

https://docs.google.com/document/d/1No4LUnsbL4L_HJG06SrHcRSpmex2oYBvbLKFrKjD2RU/edit

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks like you have this URL duplicated.

@orangemocha

Copy link
Copy Markdown
Contributor

Rubber stamp LGTM

Minor suggestion: I would expand "f-t-f" to "face-to-face VM summit" because it might not be obvious to everyone what f-t-f stands for. Could also add a link to https://github.com/jasnell/vm-summit

@mhdawson

mhdawson commented Apr 28, 2016 •

Copy link
Copy Markdown
Member Author

@trevnorris @orangemocha addressed comments.

Comment thread wg-meetings/2016-04-21.md Outdated

## Round Table

Went through stand up, mostly participation in vm f-to-f and then

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Still says "f-to-f"

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Sorry fixed title, but should have done more thourough search.

@mhdawson

Copy link
Copy Markdown
Member Author

@orangemocha I think I got them all this time.

@orangemocha

Copy link
Copy Markdown
Contributor

LGTM

@ianwjhalliday

Copy link
Copy Markdown

A link to the YouTube recording would be handy

Comment thread wg-meetings/2016-04-21.md Outdated
* Stefan Budeanu @stefanmb
* Josh Gavant
* Jianchun
* Alexis

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Fill in other names?

Josh Gavant @joshgav
Jianchun Xu @jianchun
Alexis Campailla @orangemocha

@mhdawson

mhdawson commented May 6, 2016

Copy link
Copy Markdown
Member Author

@ianwjhalliday addressed your comments. If I can get your ok I'll go ahead and land.

@ianwjhalliday

Copy link
Copy Markdown

👍

mhdawson added a commit that referenced this pull request May 9, 2016
PR-URL: #19
Reviewed-by: Alexis Campailla <alexis@janeasystems.com>
Reviewed-by: Ian Halliday <Ian.Halliday@microsoft.com>
@mhdawson

mhdawson commented May 9, 2016

Copy link
Copy Markdown
Member Author

Landed as 65b55ee

@mhdawson mhdawson self-assigned this May 9, 2016
@mhdawson mhdawson closed this May 9, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants