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: add support info #843

Merged
merged 2 commits into from
Dec 1, 2020
Merged

doc: add support info #843

merged 2 commits into from
Dec 1, 2020

Conversation

mhdawson
Copy link
Member

Copy link
Member

@NickNaso NickNaso left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@KevinEady KevinEady left a comment

Choose a reason for hiding this comment

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

According to Integration into package.json, it says:

In all of these cases package-support.json (or your alternative filename) should be included in your files array, or .npmignore configured to avoid excluding it.

Should this be done on this PR as well...?

{
"version": "*",
"target": {
"node": "lts"
Copy link
Contributor

Choose a reason for hiding this comment

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

According to node namespace, should this be active? Since we support node 10,12,14,15 (whereas lts is only 10,12)

Copy link
Member Author

Choose a reason for hiding this comment

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

LTS is now 10,12,14, but regardless good point.

Copy link
Member Author

Choose a reason for hiding this comment

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

@KevinEady fixed.

@mhdawson mhdawson merged commit d9e11ff into nodejs:master Dec 1, 2020
kevindavies8 added a commit to kevindavies8/node-addon-api-Develop that referenced this pull request Aug 24, 2022
* doc: add support info

PR-URL: nodejs/node-addon-api#843
Reviewed-By: Chengzhong Wu <[email protected]>
Reviewed-By: NickNaso <[email protected]>
Marlyfleitas added a commit to Marlyfleitas/node-api-addon-Development that referenced this pull request Aug 26, 2022
* doc: add support info

PR-URL: nodejs/node-addon-api#843
Reviewed-By: Chengzhong Wu <[email protected]>
Reviewed-By: NickNaso <[email protected]>
wroy7860 added a commit to wroy7860/addon-api-benchmark-node that referenced this pull request Sep 19, 2022
* doc: add support info

PR-URL: nodejs/node-addon-api#843
Reviewed-By: Chengzhong Wu <[email protected]>
Reviewed-By: NickNaso <[email protected]>
johnfrench3 pushed a commit to johnfrench3/node-addon-api-git that referenced this pull request Aug 11, 2023
* doc: add support info

PR-URL: nodejs/node-addon-api#843
Reviewed-By: Chengzhong Wu <[email protected]>
Reviewed-By: NickNaso <[email protected]>
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