Skip to content

docs: update 8.5.0 changelog#15384

Closed
MylesBorins wants to merge 1 commit into
nodejs:masterfrom
MylesBorins:update-8.5.0-change-log
Closed

docs: update 8.5.0 changelog#15384
MylesBorins wants to merge 1 commit into
nodejs:masterfrom
MylesBorins:update-8.5.0-change-log

Conversation

@MylesBorins
Copy link
Copy Markdown
Contributor

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

refs: #15308 (comment)

@MylesBorins MylesBorins requested a review from jasnell September 13, 2017 08:52
@nodejs-github-bot nodejs-github-bot added the doc Issues and PRs related to the documentations. label Sep 13, 2017
@MylesBorins
Copy link
Copy Markdown
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
Copy Markdown
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 MylesBorins force-pushed the update-8.5.0-change-log branch from b46ad0b to 366153e Compare September 13, 2017 16:28
@MylesBorins
Copy link
Copy Markdown
Contributor Author

@BridgeAR done... please review

@BridgeAR
Copy link
Copy Markdown
Member

Thanks!

@BridgeAR
Copy link
Copy Markdown
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
Copy Markdown
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.

7 participants