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 Sep 13 2017 #349

Merged
merged 3 commits into from
Sep 18, 2017

Conversation

mhdawson
Copy link
Member

No description provided.


### nodejs/TSC
* New TSC Director/Chair Elections [#326](https://github.com/nodejs/TSC/issues/326)
* No objections, Myles acclaimed as Directory, Michael acclaimed as Chair
Copy link
Contributor

Choose a reason for hiding this comment

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

Director

Copy link
Contributor

Choose a reason for hiding this comment

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

Would it be better to have it as Michael Dawson, as we had Michaël and Michael in the meeting today?

Copy link
Member

Choose a reason for hiding this comment

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

acclaimed is the wrong verb here. (It means praised.) I think named would be a better word choice. @mhdawson Would you prefer we push nits like that ourselves, or do you prefer to handle them so you can evaluate them?

Copy link
Member

Choose a reason for hiding this comment

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

Full names or else GitHub handles seem like the way to go to avoid ambiguity.

Copy link
Member Author

Choose a reason for hiding this comment

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

I'm ok with people pushing the nits themselves.

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

* Agreed we should move forward with the proposed times.

* 2FA for the entire org [#301](https://github.com/nodejs/TSC/issues/301)
* We have consensus on step 1, will take of agenda.
Copy link
Contributor

Choose a reason for hiding this comment

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

I always get confused, is it of or off?

Copy link
Member

Choose a reason for hiding this comment

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

off

Copy link
Member Author

Choose a reason for hiding this comment

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

yup typo in this case.

* 2FA for the entire org [#301](https://github.com/nodejs/TSC/issues/301)
* We have consensus on step 1, will take of agenda.
* Myles, explore usb keys, could foundation fund ?
* Rich, is has not been an issue so far, can we defer until we see
Copy link
Contributor

Choose a reason for hiding this comment

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

it has not

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


* Candidate "areas" for TSC and Community Committee [#278](https://github.com/nodejs/TSC/issues/278)
* Michael gave overview and request to get involved in issue
* Mateo highlighted Metrics as one where we might do more.
Copy link
Contributor

Choose a reason for hiding this comment

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

Matteo

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

internal state machine.
* If we have separate object that refers to existing one, but
Matteo believes this may be an issue due to additional allocation
* Jeremiah just consider _XX for writable and readable part of
Copy link
Contributor

Choose a reason for hiding this comment

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

Nit: I think the _ has to be escaped here.

Copy link
Member Author

Choose a reason for hiding this comment

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

I looked at in "view" and it seemed ok.

* Michael gave an overview of the proposal. Wanted to make sure TSC
was comfortable moving forward.
* We agreed to leave on agenda, take discussion back to github
for 1 week and discuss in the meeting gnext week.
Copy link
Contributor

Choose a reason for hiding this comment

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

next week

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


* not seen simple user access control with react ?
* we are not sure what the questions is.
* try using the help nodejs/repo
Copy link
Contributor

Choose a reason for hiding this comment

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

Nit: nodejs/help repo

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

* V8 reports that 6.1 has a larger number of fixes that would help
ongoing maintenance
* Rod, originally we said 2 months at one point
* We would need to set end Oct at LTS release date
Copy link
Contributor

Choose a reason for hiding this comment

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

Nit: as LTS release date?

@mhdawson
Copy link
Member Author

pushed update to address comments so far.


### nodejs/TSC
* New TSC Director/Chair Elections [#326](https://github.com/nodejs/TSC/issues/326)
* No objections, Myles Borrins named as Director, Michael Dawson named as Chair
Copy link
Contributor

Choose a reason for hiding this comment

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

Myles Borins

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

@jasnell jasnell merged commit a17bfc7 into nodejs:master Sep 18, 2017
@mhdawson mhdawson deleted the minutesSep13 branch August 14, 2019 16:58
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