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.

Sign up for GitHub

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

Jump to bottom

doc: add minutes for Dec 2, 2016 #10

Merged
merged 1 commit into from
Jan 5, 2017
Merged

doc: add minutes for Dec 2, 2016 #10

merged 1 commit into from
Jan 5, 2017

Conversation

sam-github
Copy link
Contributor

Based on my notes.

Please comment and I will add comments to the notes.

If anyone has better records of attendance, that would be great to have.

@sam-github sam-github mentioned this pull request Dec 13, 2016
Closed
@joshgav
Copy link
Contributor

joshgav commented Dec 13, 2016

My notes, with attendees too: https://docs.google.com/document/d/1-gY_sNt4CgySEale_13e2n9S9ucjuoM7djCq3Njal-0

@sam-github
Copy link
Contributor Author

Added @joshgav's notes.

@sam-github
Copy link
Contributor Author

LGTM from anyone?

@sam-github sam-github mentioned this pull request Dec 22, 2016
Closed
MylesBorins
MylesBorins approved these changes Dec 30, 2016
View reviewed changes
Copy link

@MylesBorins MylesBorins left a comment

Choose a reason for hiding this comment

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

LGTM

Will land EOD if no objections

@vdeturckheim
Copy link
Member

LGTM

1 similar comment
@mcollina
Copy link
Member

LGTM

mhdawson
mhdawson approved these changes Jan 3, 2017
View reviewed changes
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

@sam-github sam-github merged commit d09ae5c into nodejs:master Jan 5, 2017
patrickm68 added a commit to patrickm68/security-wg-process that referenced this pull request Sep 14, 2023
@patrickm68
doc: add minutes for Dec 2, 2016
a04af42 
PR-URL: nodejs/security-wg#10
Reviewed-By: Myles Borins <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: Sam Roberts <[email protected]>
Reviewed-By: Matteo Collina <[email protected]>
mattstern31 added a commit to mattstern31/security-wg-process that referenced this pull request Nov 11, 2023
@mattstern31
doc: add minutes for Dec 2, 2016
18f05ba 
PR-URL: nodejs/security-wg#10
Reviewed-By: Myles Borins <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: Sam Roberts <[email protected]>
Reviewed-By: Matteo Collina <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mhdawson mhdawson mhdawson approved these changes

@MylesBorins MylesBorins MylesBorins approved these changes

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
@sam-github @joshgav @vdeturckheim @mcollina @MylesBorins @mhdawson