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

docs: update 8.5.0 changelog #15384

Closed

Conversation

MylesBorins
Copy link
Contributor

The original changelog included incorrect information regarding
the new perf_hooks api.

refs: #15308 (comment)

@nodejs-github-bot nodejs-github-bot added the doc Issues and PRs related to the documentations. label Sep 13, 2017
@MylesBorins
Copy link
Contributor Author

I don't think this needs to wait 48 hours. If there are no objections I'll be landing it in about 2 hours

@BridgeAR
Copy link
Member

@MylesBorins would you be so kind and also change https://github.com/nodejs/node/blame/v8.5.0/doc/api/assert.md#L108 to https://github.com/nodejs/node/pull/15001 while being on it? That PR landed with the wrong PR number.

The original changelog included incorrect information regarding
the new perf_hooks api.

refs: nodejs#15308 (comment)
@MylesBorins
Copy link
Contributor Author

@BridgeAR done... please review

@BridgeAR
Copy link
Member

Thanks!

@BridgeAR
Copy link
Member

I do not think that this has to wait the regular 48 hours.

MylesBorins added a commit that referenced this pull request Sep 14, 2017
The original changelog included incorrect information regarding
the new perf_hooks api.

refs: #15308 (comment)

PR-URL: #15384
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Michaël Zasso <mic.besace@gmail.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
@MylesBorins
Copy link
Contributor Author

landed in cba206f

MylesBorins added a commit that referenced this pull request Sep 14, 2017
The original changelog included incorrect information regarding
the new perf_hooks api.

refs: #15308 (comment)

PR-URL: #15384
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Michaël Zasso <mic.besace@gmail.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
addaleax pushed a commit to addaleax/ayo that referenced this pull request Sep 17, 2017
The original changelog included incorrect information regarding
the new perf_hooks api.

refs: nodejs/node#15308 (comment)

PR-URL: nodejs/node#15384
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Michaël Zasso <mic.besace@gmail.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Qard pushed a commit to Qard/ayo that referenced this pull request Sep 21, 2017
The original changelog included incorrect information regarding
the new perf_hooks api.

refs: nodejs/node#15308 (comment)

PR-URL: nodejs/node#15384
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Michaël Zasso <mic.besace@gmail.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
@MylesBorins MylesBorins deleted the update-8.5.0-change-log branch November 14, 2017 17:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc Issues and PRs related to the documentations.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants