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: fix nits in blog/release/v8.0.0.md #1244

Merged
merged 1 commit into from
May 31, 2017
Merged

doc: fix nits in blog/release/v8.0.0.md #1244

merged 1 commit into from
May 31, 2017

Conversation

vsemozhetbyt
Copy link
Contributor

@vsemozhetbyt vsemozhetbyt commented May 30, 2017

Still not sure about this.

Copy link
Member

@Trott Trott left a comment

Choose a reason for hiding this comment

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

LGTM

@@ -200,12 +200,12 @@ AssertionError [ERR_ASSERTION]: false == true

Information for static error codes can be queried quickly by referencing the
Node.js documentation. For instance, the URL to lookup information about the
`ERR_ASSERTION` error code is
https://nodejs.org/dist/latest-v7.x/docs/api/errors.html#ERR_ASSERTION.
`ERR_ARG_NOT_ITERABLE` error code is
Copy link
Member

@jasnell jasnell May 30, 2017

Choose a reason for hiding this comment

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

I could have sworn that ERR_ASSERTION had been added... will have to make sure it is.

The link format does work tho... try

https://nodejs.org/dist/latest-v8.x/docs/api/errors.html#ERR_ARG_NOT_ITERABLE

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done)

@vsemozhetbyt
Copy link
Contributor Author

Should this wait 48 hours?

@lpinca
Copy link
Member

lpinca commented May 31, 2017

That rule does not apply to the website but let's wait a little more to give other collaborators a chance to review.

@lpinca lpinca merged commit 47dae00 into nodejs:master May 31, 2017
@vsemozhetbyt vsemozhetbyt deleted the v8.0.0.md branch May 31, 2017 16:49
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