-
Notifications
You must be signed in to change notification settings - Fork 29.7k
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: fix doc for napi_create_function #21627
doc: fix doc for napi_create_function #21627
Conversation
Node.js Collaborators, please, add 👍 here if you approve fast-tracking. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Resumed build as: https://ci.nodejs.org/job/node-test-pull-request/15742/ |
Another attempt at resuming the build: https://ci.nodejs.org/job/node-test-pull-request/15743/ |
Merge the two duplicate entries for `napi_create_function()`. Fixes: nodejs#21606
084da9b
to
6131981
Compare
This one actually finished: https://ci.nodejs.org/job/node-test-pull-request/15745/ |
Landed in e55bdde. |
Merge the two duplicate entries for `napi_create_function()`. Fixes: nodejs#21606 PR-URL: nodejs#21627 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com> Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Merge the two duplicate entries for
napi_create_function()
.Fixes: #21606
Checklist
make -j4 test
(UNIX), orvcbuild test
(Windows) has unrelated errors 😕