-
-
Notifications
You must be signed in to change notification settings - Fork 393
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
N-API Support #271
Comments
Anyone care to provide an outline of what would be required to implement support for N-API? |
Any plans on this? It gonna save tons of time doing re-compilation especially for electron apps. |
@JoshuaWise would you be open to a PR that ported the V8 interface to N-API? I've been noodling on the continuous maintenance needed for Electron (#638, #693, etc.) and I believe using N-API might alleviate the need to update the build scripts every release. |
Bumping this. I'm interested in taking this on, although I believe it's worthwhile porting off |
Any update on this? |
Grretings.
First of all, thanks for all the effort creating and maintaining this module, it's very efficient and convenient to use.
I would like to know if there are any plans for supporting N-API https://nodejs.org/docs/latest-v8.x/api/n-api.html?
Possibly related issue with a lot of info in it – TryGhost/node-sqlite3#830
The text was updated successfully, but these errors were encountered: