Skip to content

add minutes for most recent meeting - #441

Merged
Trott merged 5 commits into
nodejs:masterfrom
Trott:dec-13-2017-minutes
Dec 15, 2017
Merged

Trott merged 5 commits into
nodejs:masterfrom
Trott:dec-13-2017-minutes

Conversation

@Trott

@Trott Trott commented Dec 13, 2017

Copy link
Copy Markdown
Member

No description provided.

Comment thread meetings/2017-12-13.md Outdated

* V8: Opened PR for 6.4. Not expecting major issues with this update.

* Module: There was the blog post on the

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Add:

Node.js Foundation's Medium Publication The Current State of Implementation and Planning for ESModules

Comment thread meetings/2017-12-13.md Outdated

Franzi: Move to next week.

Rich: Yes, agreed. Let’s gather votes for people who are here, though.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think this is for the above issue?

Comment thread meetings/2017-12-13.md

* events: add off alias to removeListener [#17156](https://github.com/nodejs/node/pull/17156)

Rich: GitHub conversation did not get “unstuck”. Move to vote.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

What was the outcome here? Was a vote taken? If not, when will it be taken?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

IIRC we want for the TSC members to vote in the issue. We might already have enough approvals there, let me check.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Not enough votes yet. Left a comment.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Comment thread meetings/2017-12-13.md Outdated

Franzi: Myles put it on the agenda and wanted to know if we should work on it or close it. I vote to close it since we can always re-open or open a more targeted issue.

RIch: +1

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

s/RIch/Rich

Comment thread meetings/2017-12-13.md Outdated

* V8: Opened PR for 6.4. Not expecting major issues with this update.

* Module: There was the blog post on the

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

What’s the rest of this sentence?

@fhinkel fhinkel left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM with minor nits

Comment thread meetings/2017-12-13.md Outdated

Franzi: Leave it for next week’s meeting.

Rich: Invite Bradley as a guess.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

guess --> guest?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

/cc @mhdawson to make sure we don't forget this

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@Trott volunteered to chair next week's meeting as I'm off on holiday until Jan 2 starting Friday. The issue will be created with observers, @Trott can you add him once the issue is generated (~9AM EST)

@Trott Trott Dec 14, 2017 •

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yeah, I made a note in my calendar for Monday to invite Bradley, Gibson, and Ruben as guests. I'll no doubt scan over the agenda to see if there's anyone else to invite and probably ask William Kapke too. I'll also double-check to make sure @danbev has been added to the list of attendees. I'm guessing that is something you might have to update in your cron script or whatever it is, @mhdawson.

Comment thread meetings/2017-12-13.md

* events: add off alias to removeListener [#17156](https://github.com/nodejs/node/pull/17156)

Rich: GitHub conversation did not get “unstuck”. Move to vote.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Comment thread meetings/2017-12-13.md Outdated

Franzi: Leave it for next week’s meeting.

Rich: Invite Bradley as a guess.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

/cc @mhdawson to make sure we don't forget this

Comment thread meetings/2017-12-13.md

* build: update minimum kernel version to 3.10.0 [#14795](https://github.com/nodejs/node/pull/14795)

Rich: Given this and the last issue, maybe we should invite Ruben to the meeting as an observer too next time to frame these issues.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

/cc @mhdawson to remember to invite ruben

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@Trott same comment as last time if you can add Ruben to next week's meeting.

Comment thread meetings/2017-12-13.md

Rich: Given this and the last issue, maybe we should invite Ruben to the meeting as an observer too next time to frame these issues.

Franzi: Agreed. We need more people to decide and frame these. Maybe we should invite Gibson too.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

also + gibson

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@Trott ditto

@mhdawson mhdawson left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@Trott
Trott merged commit fa45577 into nodejs:master Dec 15, 2017
@Trott
Trott deleted the dec-13-2017-minutes branch March 11, 2021 14:36
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.

7 participants