Skip to content

Conversation

@JungMinu
Copy link
Member

@JungMinu JungMinu commented Nov 4, 2016

Please review this post. ❤️,@nodejs/evangelism

Please review this post. ❤️,@nodejs/evangelism
@Knighton910
Copy link
Contributor

LGTM 🐧


### Node v7.1.0 proposal

* [Node v7.1.0 proposal](https://github.com/nodejs/node/pull/9438). The complete changelog from previous releases can be found [on GitHub](https://github.com/nodejs/node/blob/master/CHANGELOG.md).
Copy link
Member

Choose a reason for hiding this comment

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

Nit: I would leave out from previous releases and just write The complete changelog can be found...

### Community Updates

* [Node.js v7 Release Brief](https://nodesource.com/blog/node-js-v7-release-brief)
* [Last Week in Node.js Working Groups](https://nodesource.com/blog/last-week-in-node-js-working-groups-october-17-24): Last week was quieter than usual as a lot of effort was put into the new v7 Current release line, as such, this is the summary for the last two weeks.
Copy link
Member

Choose a reason for hiding this comment

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

Nit: Fix comma splice: ...Current release line. As such, this is...

@Trott
Copy link
Member

Trott commented Nov 4, 2016

Some small nits, but LGTM with or without them corrected.

@JungMinu
Copy link
Member Author

JungMinu commented Nov 5, 2016

@Trott All nits addressed and updated :)

@JungMinu JungMinu merged commit acd44f2 into master Nov 5, 2016
@JungMinu JungMinu deleted the JungMinu-weekly-news branch November 5, 2016 01:39
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.

5 participants