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

docs: meeting minutes 2017-03-23 #229

Merged
merged 1 commit into from
May 30, 2017
Merged

docs: meeting minutes 2017-03-23 #229

merged 1 commit into from
May 30, 2017

Conversation

joshgav
Copy link
Contributor

@joshgav joshgav commented Mar 24, 2017

cc @nodejs/tsc @mikeal

* William Kapke @williamkapke (observer/NF)
* Michael Dawson @mhdawson (observer/CTC)
* Anna Henningsen @addaleax (observer/CTC)
* Jenn Turner @renrutnnej (observer/Linux Foundation)
Copy link
Member

Choose a reason for hiding this comment

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

You forgot a certain dashing young programmer...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed!


@jasnell: Have not had to exercise this, don’t anticipate issues.

Conclusion: Go with simple majority/>50% for everything to minimize friction.
Copy link
Member

Choose a reason for hiding this comment

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

...and removal after three months of inactivity, not six.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated, PTAL and make sure I captured this correctly. Thanks!

@jasnell
Copy link
Member

jasnell commented Apr 17, 2017

Ping @joshgav ... PTAL at @bnoordhuis' comments

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 subject to changes requested by Ben.

@joshgav
Copy link
Contributor Author

joshgav commented Apr 20, 2017

Updated conclusion of membership resignation discussion per Ben's comments above, PTAL to be sure I got it right. Thanks!

PR-URL: #229
Reviewed-By: _tbd_
Reviewed-By: _tbd_
@jasnell jasnell dismissed bnoordhuis’s stale review May 30, 2017 03:33

Updated to resolve the issue

@jasnell jasnell merged commit 9939962 into nodejs:master May 30, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants