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

Updade Node-API version matrix #48277

Closed
gabrielschulhof opened this issue Jun 1, 2023 · 2 comments
Closed

Updade Node-API version matrix #48277

gabrielschulhof opened this issue Jun 1, 2023 · 2 comments
Labels
doc Issues and PRs related to the documentations. node-api Issues and PRs related to the Node-API.

Comments

@gabrielschulhof
Copy link
Contributor

Version

main

Platform

all

Subsystem

node-api

What steps will reproduce the bug?

After we release and backport Version 9, we should update the support matrix with the Node.js versions that provide it, and we should also correct the document to say that opting into versions > 8 requires the presence of #define NAPI_VERSION n where n is the version > 8 required.

How often does it reproduce? Is there a required condition?

100%

What is the expected behavior? Why is that the expected behavior?

Doc is correct.

What do you see instead?

See recommended changes above.

Additional information

No response

@gabrielschulhof gabrielschulhof added doc Issues and PRs related to the documentations. node-api Issues and PRs related to the Node-API. labels Jun 1, 2023
@gabrielschulhof gabrielschulhof changed the title Upade Node-API version matrix Updade Node-API version matrix Jun 16, 2023
mhdawson added a commit to mhdawson/io.js that referenced this issue Jul 14, 2023
- refactor table so that it does not grow for
  every new version of Node.js
- refer to latest version of table as the canonical
  reference
- add info for Node-API v9

Refs: nodejs#48277

Signed-off-by: Michael Dawson <[email protected]>
@mhdawson
Copy link
Member

PR to address - #48774. Also refactored the table so that it takes less space needs fewer updates.

nodejs-github-bot pushed a commit that referenced this issue Jul 19, 2023
- refactor table so that it does not grow for
  every new version of Node.js
- refer to latest version of table as the canonical
  reference
- add info for Node-API v9

Refs: #48277

Signed-off-by: Michael Dawson <[email protected]>
PR-URL: #48774
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
@mhdawson
Copy link
Member

matrix is updated, closing

pluris pushed a commit to pluris/node that referenced this issue Aug 6, 2023
- refactor table so that it does not grow for
  every new version of Node.js
- refer to latest version of table as the canonical
  reference
- add info for Node-API v9

Refs: nodejs#48277

Signed-off-by: Michael Dawson <[email protected]>
PR-URL: nodejs#48774
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
pluris pushed a commit to pluris/node that referenced this issue Aug 7, 2023
- refactor table so that it does not grow for
  every new version of Node.js
- refer to latest version of table as the canonical
  reference
- add info for Node-API v9

Refs: nodejs#48277

Signed-off-by: Michael Dawson <[email protected]>
PR-URL: nodejs#48774
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Ceres6 pushed a commit to Ceres6/node that referenced this issue Aug 14, 2023
- refactor table so that it does not grow for
  every new version of Node.js
- refer to latest version of table as the canonical
  reference
- add info for Node-API v9

Refs: nodejs#48277

Signed-off-by: Michael Dawson <[email protected]>
PR-URL: nodejs#48774
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Ceres6 pushed a commit to Ceres6/node that referenced this issue Aug 14, 2023
- refactor table so that it does not grow for
  every new version of Node.js
- refer to latest version of table as the canonical
  reference
- add info for Node-API v9

Refs: nodejs#48277

Signed-off-by: Michael Dawson <[email protected]>
PR-URL: nodejs#48774
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
UlisesGascon pushed a commit to UlisesGascon/node that referenced this issue Aug 14, 2023
- refactor table so that it does not grow for
  every new version of Node.js
- refer to latest version of table as the canonical
  reference
- add info for Node-API v9

Refs: nodejs#48277

Signed-off-by: Michael Dawson <[email protected]>
PR-URL: nodejs#48774
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
RafaelGSS pushed a commit that referenced this issue Aug 15, 2023
- refactor table so that it does not grow for
  every new version of Node.js
- refer to latest version of table as the canonical
  reference
- add info for Node-API v9

Refs: #48277

Signed-off-by: Michael Dawson <[email protected]>
PR-URL: #48774
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
targos pushed a commit that referenced this issue Nov 27, 2023
- refactor table so that it does not grow for
  every new version of Node.js
- refer to latest version of table as the canonical
  reference
- add info for Node-API v9

Refs: #48277

Signed-off-by: Michael Dawson <[email protected]>
PR-URL: #48774
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
sercher added a commit to sercher/graaljs that referenced this issue Apr 25, 2024
- refactor table so that it does not grow for
  every new version of Node.js
- refer to latest version of table as the canonical
  reference
- add info for Node-API v9

Refs: nodejs/node#48277

Signed-off-by: Michael Dawson <[email protected]>
PR-URL: nodejs/node#48774
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
sercher added a commit to sercher/graaljs that referenced this issue Apr 25, 2024
- refactor table so that it does not grow for
  every new version of Node.js
- refer to latest version of table as the canonical
  reference
- add info for Node-API v9

Refs: nodejs/node#48277

Signed-off-by: Michael Dawson <[email protected]>
PR-URL: nodejs/node#48774
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
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. node-api Issues and PRs related to the Node-API.
Projects
None yet
Development

No branches or pull requests

2 participants