-
-
Notifications
You must be signed in to change notification settings - Fork 31.4k
doc: fix doc for napi_create_function #21627
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
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 😕