◐ Shell
clean mode source ↗

[v10.x] doc,n-api: update matrix for N-API version 4 by richardlau · Pull Request #27832 · nodejs/node

Skip to content

Navigation Menu

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Conversation

@richardlau

The N-API version was bumped to 4 for Node.js 10 in
#25633

Refs: #25633

This should land in the release with the above referenced PR.

cc @nodejs/lts @nodejs/n-api

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • documentation is changed or added
  • commit message follows commit guidelines

@nodejs-github-bot

@richardlau

Once this lands in a Node.js 10.x release (i.e. the actual version of Node.js 10 that first supports N-API version 4 is known) a follow up PR needs to add that information to the N-API docs on master.

@mhdawson

@richardlau I assume the REPLACEME's are handled as part of the release process right?

@richardlau

BethGriggs

codebytere

mhdawson

Choose a reason for hiding this comment

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

LGTM

@BethGriggs

BethGriggs pushed a commit that referenced this pull request

May 28, 2019
The N-API version was bumped to 4 for Node.js 10 in
#25633

Refs: #25633

PR-URL: #27832
Reviewed-By: Beth Griggs <Bethany.Griggs@uk.ibm.com>
Reviewed-By: Shelley Vohr <codebytere@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>

Labels

doc

Issues and PRs related to the documentations.

node-api

Issues and PRs related to the Node-API.