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 minutes for meeting Oct 25 2017 #398

Closed
wants to merge 4 commits into from

Conversation

mhdawson
Copy link
Member

No description provided.


* Update from the moderation team (Forrest)

* Deck used for persentation https://docs.google.com/presentation/d/1cN712pneaa7L6_wVl9-M2OpK_2Am8EbarYyoJxxyyYA/edit#slide=id.p
Copy link
Contributor

Choose a reason for hiding this comment

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

presentation

Copy link
Member Author

Choose a reason for hiding this comment

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

fixed.

### nodejs/node

* Support both OpenSSL 1.1.0 and 1.0.2 [#16130](https://github.com/nodejs/node/pull/16130)
* current state is that is is getting ready to land
Copy link
Contributor

Choose a reason for hiding this comment

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

it is?

Copy link
Member Author

Choose a reason for hiding this comment

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

fixed

@mhdawson
Copy link
Member Author

Pushed changes to address comments. @nodejs/tsc please review.


* Deck used for presentation https://docs.google.com/presentation/d/1cN712pneaa7L6_wVl9-M2OpK_2Am8EbarYyoJxxyyYA/edit#slide=id.p

* 10 members, so far have had to be intervene in 0 zeros, though there were 2 request
Copy link
Contributor

Choose a reason for hiding this comment

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

What is 0 zeros? Also, request -> requests.

Copy link
Member Author

Choose a reason for hiding this comment

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

Fixed, believe if should have been issues. Added s to request.

* team not working group
* scope set as enforcing policies, not setting them.
* With thefourtheye joining the team now has global timezone coverage
(this has been challenge due to being a small team)
Copy link
Contributor

Choose a reason for hiding this comment

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

There are a few extra spaces after "been"

Copy link
Member Author

Choose a reason for hiding this comment

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

Fixed


## Q&A, Other

* From irc, Bryan, number of uncharted WG's not an issue to have groups to
Copy link
Contributor

Choose a reason for hiding this comment

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

Is it worth adding Bryan's last name? I'm not sure that it matters.

Copy link
Member Author

Choose a reason for hiding this comment

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

Added Bryan's last name.

@mhdawson mhdawson closed this Nov 2, 2017
@mhdawson mhdawson reopened this Nov 2, 2017
mhdawson added a commit that referenced this pull request Nov 2, 2017
PR-URL: #398
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
@mhdawson
Copy link
Member Author

mhdawson commented Nov 2, 2017

Landed as b585c06

@mhdawson mhdawson closed this Nov 2, 2017
@mhdawson mhdawson deleted the minutesOct25 branch August 14, 2019 17:00
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