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 2017-06-05 #227

Merged
merged 1 commit into from
Jun 6, 2017

Conversation

gibfahn
Copy link
Member

@gibfahn gibfahn commented Jun 5, 2017

Closes: #225

@nodejs/lts, there was a fair amount of discussion on this one, so please make sure I didn't misrepresent anyone.

If you have any changes to make just click the edit button and add commits using the GitHub interface.

@gibfahn gibfahn force-pushed the june-minutes branch 2 times, most recently from 451715d to 6a7f50f Compare June 5, 2017 22:50
gibfahn added a commit to gibfahn/Release that referenced this pull request Jun 5, 2017
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

- All: Let’s take this bikeshed offline.
- Jeremiah: WRT naming, I think the key thing is to make it clear for outsiders
to our process (what does this WG actually mean?)
- Sam: ^---- agree with Jeremiah on clarity for outsiders. LTS what makes a
Copy link
Contributor

Choose a reason for hiding this comment

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

s/LTS what/Which is what/

Copy link
Contributor

@sam-github sam-github left a comment

Choose a reason for hiding this comment

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

one nit, otherwise LGTM

@gibfahn gibfahn merged commit 2b4253c into nodejs:master Jun 6, 2017
@gibfahn gibfahn deleted the june-minutes branch June 6, 2017 16:23
ChALkeR pushed a commit to ChALkeR/LTS that referenced this pull request Jun 30, 2018
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