Skip to content

Conversation

RaisinTen
Copy link
Member

This was asked for in nodejs/node-addon-api#110 (comment) and nodejs/node-addon-api#110 (comment). The solution was described in text in nodejs/node-addon-api#110 (comment) but it would be better to have a code example that folks can refer to.

cc @mhdawson @NickNaso

This was asked for in
nodejs/node-addon-api#110 (comment)
and
nodejs/node-addon-api#110 (comment).
The solution was described in text in
nodejs/node-addon-api#110 (comment)
but it would be better to have a code example that folks can refer to.

Signed-off-by: Darshan Sen <raisinten@gmail.com>
Copy link
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 ebcb9fe into nodejs:main Jun 28, 2023
@RaisinTen RaisinTen deleted the add-example-for-calling-js-from-AsyncWorker-Execute branch June 30, 2023 07:09
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.

2 participants