-
Notifications
You must be signed in to change notification settings - Fork 459
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
Add API for working with Promises #126
Comments
A C API for promises was just recently implemented at nodejs/node#14365 Now it definitely makes sense to expose that in the C++ wrapper. @gabrielschulhof are you planning to do that? Otherwise I might get to it eventually. Or someone else interested may pick it up. |
This issue should be closed now. |
@JamesMGreene It would be great to have some docs/examples ( |
@koistya We are working on documentation. What do you think if we add the example or the references to the examples inside the documentation? |
Hi @koistya, |
I'm closing the issue because now the API to work with Promise has been added and documented. |
How hard would it be to implement APIs allowing writing Promise-based N-API addons?
The text was updated successfully, but these errors were encountered: