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 29 Nov 2017 #430

Merged
merged 3 commits into from
Dec 5, 2017
Merged

Conversation

mhdawson
Copy link
Member

No description provided.

### nodejs/node

* Enable explicit `.m.js` intent for ESM [#16170](https://github.com/nodejs/node/pull/16170)
* Bradley, that thread contains the thought that we don.t need to import non-esm modules.
Copy link
Contributor

Choose a reason for hiding this comment

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

don.t --> don't

* Bradley, that thread contains the thought that we don.t need to import non-esm modules.
There are a number so should likely be more generic to handle others like wasm,
json etc. 2 older formats that should be included in the discussion as well.
* Next action action, Myles to schedule working session which includes this and other
Copy link
Contributor

Choose a reason for hiding this comment

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

Next action action --> Next action

There are a number so should likely be more generic to handle others like wasm,
json etc. 2 older formats that should be included in the discussion as well.
* Next action action, Myles to schedule working session which includes this and other
topics that we don.t have consensus on.
Copy link
Contributor

Choose a reason for hiding this comment

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

don.t --> don't

* Current set interval has callback that takes a significant amount of time, next interval will
be scheduled after the callback end. PR changes to schedule so that it is based on
when the previous one started.
* Light discussion in the TSE meeting. New behaviour sounds right (needs to be
Copy link
Contributor

Choose a reason for hiding this comment

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

TSE --> TSC

* Bradley - suggesting we make the controller the Foundation as opposed to the TSC.
* James - should be ok if we tie to the TSC. Does not need to be legal entity.
* Bradley - they are asking for .standards. org. That is what should be the Foundation
* James - don.t think it matters.
Copy link
Contributor

Choose a reason for hiding this comment

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

don.'t --> don't

* James - don.t think it matters.
* Bradley, leave TSC as change controller and standard org is identified as Foundation.
* Rod - rather have the TSC as both.
* We.ll move forward with it as the TSC.
Copy link
Contributor

Choose a reason for hiding this comment

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

We.ll --> We'll

* MichaëZasso @targos (TSC)
* Rod Vagg @rvagg (TSC)
* Sakthipriyan Vairamani @thefourtheye (TSC)
* Bradley Meck @bmeck (Obserer)
Copy link
Contributor

Choose a reason for hiding this comment

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

bmeck is mentioned twice

@mhdawson
Copy link
Member Author

@maclover7 thanks for the comments, @Trott thanks for the fixes :)

@Trott Trott merged commit 258e75f into nodejs:master Dec 5, 2017
@mhdawson mhdawson deleted the minutes29nov 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.

3 participants