-
Notifications
You must be signed in to change notification settings - Fork 29.1k
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
There may be a syntax error in the nodejs napi doc. #21049
Labels
doc
Issues and PRs related to the documentations.
good first issue
Issues that are suitable for first-time contributors.
node-api
Issues and PRs related to the Node-API.
Comments
@nodejs/n-api |
Trott
added
node-api
Issues and PRs related to the Node-API.
doc
Issues and PRs related to the documentations.
labels
May 31, 2018
vsemozhetbyt
added
the
good first issue
Issues that are suitable for first-time contributors.
label
May 31, 2018
@caijw It's not a syntax error, but a grammatical one. Feel free to publish a patch for this. |
May I address this issue? |
ohbarye
added a commit
to ohbarye/node
that referenced
this issue
May 31, 2018
just replaced `than` with `that`. - wrong: the N-API functions than can be used to change - correct: the N-API functions that can be used to change Fixes: nodejs#21049
addaleax
pushed a commit
that referenced
this issue
Jun 1, 2018
just replaced `than` with `that`. - wrong: the N-API functions than can be used to change - correct: the N-API functions that can be used to change Fixes: #21049 PR-URL: #21060 Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
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.
good first issue
Issues that are suitable for first-time contributors.
node-api
Issues and PRs related to the Node-API.
https://nodejs.org/dist/latest-v10.x/docs/api/n-api.html#n_api_object_lifetime_management
This document may has a syntax error:
In many cases, however, it is necessary that the handles remain valid for either a shorter or longer lifespan than that of the native method. The sections which follow describe the N-API functions than can be used to change the handle lifespan from the default.
...the N-API functions than can be used to change the..., than shoul change to that.
The text was updated successfully, but these errors were encountered: