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.

Sign up for GitHub

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

Jump to bottom

doc: correct grammar in README.md #8892

Closed
wants to merge 2 commits into from
Closed

doc: correct grammar in README.md #8892

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
564a97e
Update README.md
NoahDragon Oct 2, 2016
ff11d6a
Update README.md
NoahDragon Oct 2, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ source libraries in the world.

The Node.js project is supported by the
[Node.js Foundation](https://nodejs.org/en/foundation/). Contributions,
policies and releases are managed under an
policies, and releases are managed under an
[open governance model](./GOVERNANCE.md). We are also bound by a
[Code of Conduct](./CODE_OF_CONDUCT.md).

Expand Down Expand Up @@ -44,7 +44,7 @@ The Node.js project maintains multiple types of releases:
versioned by [SemVer](http://semver.org/) and signed by a member of the
[Release Team](#release-team).
Code for Current releases is organized in this repository by major version
number, For example: [v4.x](https://github.com/nodejs/node/tree/v4.x).
number, For example, [v4.x](https://github.com/nodejs/node/tree/v4.x).
Copy link
Contributor

Choose a reason for hiding this comment

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

I think changing the preceding comma to a period would be a better change here:

number. For example: ...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@mscdex Agree.

The major version number of Current releases will increment every 6 months
allowing for breaking changes to be introduced. This happens in April and
October every year. Current release lines beginning in October each year have
Expand Down Expand Up @@ -73,7 +73,7 @@ Binaries, installers, and source tarballs are available at
<https://nodejs.org/download/release/>, listed under their version strings.
The [latest](https://nodejs.org/download/release/latest/) directory is an
alias for the latest Current release. The latest LTS release from an LTS
line is available in the form: latest-_codename_. For example:
line is available in the form: latest-_codename_. For example,
Copy link
Contributor

Choose a reason for hiding this comment

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

IMHO colon looks better here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@mscdex Agree. I will make the change.

<https://nodejs.org/download/release/latest-argon>

**Nightly** builds are available at
Expand Down