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 24 2018 #616

Merged
merged 4 commits into from
Oct 31, 2018

Conversation

mhdawson
Copy link
Member

No description provided.

* Behaviour is inconsistent with the documentation.
* Read and Write behave differently
* PR exists to update documentation
* Call for people to get involved in the issue.
Copy link
Contributor

Choose a reason for hiding this comment

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

You have summarized this problem perfectly. 👏


* Strategic Initiatives - Tracking Issue [#423](https://github.com/nodejs/TSC/issues/423)
* N-API
* Getting ready for workshopp at NodeConfEU
Copy link
Contributor

Choose a reason for hiding this comment

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

Typo: workshop

* Tracking issue for updating TSC on Board Meetings [#476](https://github.com/nodejs/TSC/issues/476)
* No Myles, skipped for this week


Copy link
Contributor

Choose a reason for hiding this comment

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

Nit: We don't need these many empty lines, right?

Copy link
Member Author

Choose a reason for hiding this comment

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

Looked in the file, not sure while more spaces are shown as there was only one. Tried deleting/re-adding a single blank line.

Copy link
Member Author

Choose a reason for hiding this comment

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

Ah was looking at version without one of the commits.

@cjihrig
Copy link
Contributor

cjihrig commented Oct 25, 2018

The filename seems wrong to me: 2018-11-24.md (should be 2018-10-24).

@mhdawson
Copy link
Member Author

@cjihrig good catch, fixed.

@mhdawson mhdawson merged commit 85a7781 into nodejs:master Oct 31, 2018
@mhdawson mhdawson deleted the minutes24Oct branch August 14, 2019 16:59
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