Skip to content

doc: update doc references from N-API to Node-API#176

Merged
mhdawson merged 2 commits into
nodejs:mainfrom
xozitu:n-api-to-node-api
Mar 1, 2021
Merged

doc: update doc references from N-API to Node-API#176
mhdawson merged 2 commits into
nodejs:mainfrom
xozitu:n-api-to-node-api

Conversation

@jschlight
Copy link
Copy Markdown
Contributor

@jschlight jschlight commented Feb 17, 2021

The PR changes just the documentation including the Node-API Resource website.

See nodejs/abi-stable-node#420

@jschlight jschlight changed the title doc: Update doc references from N-API to Node-API doc: update doc references from N-API to Node-API Feb 18, 2021
Comment thread README.md Outdated
- [`nan`](https://github.com/nodejs/nan): C++-based abstraction between Node and direct V8 APIs.
- [`N-API`](https://nodejs.org/api/n-api.html): C-based API guaranteeing [ABI stability](https://nodejs.org/en/docs/guides/abi-stability/) across different node versions as well as JavaScript engines.
- [`node-addon-api`](https://github.com/nodejs/node-addon-api): header-only C++ wrapper classes which simplify the use of the C-based N-API.
- [`Node-API`](https://nodejs.org/api/n-api.html): C-based API guaranteeing [ABI stability](https://nodejs.org/en/docs/guides/abi-stability/) across different node versions as well as JavaScript engines.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Maybe add previously known as n-api ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I've added the reference to N-API.

Copy link
Copy Markdown
Member

@mhdawson mhdawson left a comment

Choose a reason for hiding this comment

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

LGTM

@mhdawson mhdawson merged commit 82e0e96 into nodejs:main Mar 1, 2021
@jschlight jschlight deleted the n-api-to-node-api branch March 1, 2021 21:32
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.

3 participants