-
Notifications
You must be signed in to change notification settings - Fork 20
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 #6
Comments
Hello again, thanks for this Nick, like with your similar improvements for |
Hi @NickNaso, I'd be happy to accept and merge a PR for this if it's something you're still interested in. |
Hi @lovell it could be great, but before to make a PR I want update and review my work. I will try to do tomorrow. |
Brilliant, thank you, there's no time pressure. |
v3.0.0 now available, thanks again for the PR. |
Hi @lovell ,
I worked on this native addon and I ported it to N-API
You can find my work here: https://github.com/NickNaso/highwayhash/tree/napi
If you want experiment with N-API you can create a branch and call it "napi" so I can execute the PR on it.
After that we can iterate to over it and at the end publish a tagged version of highwayhash like reported here: https://nodejs.org/en/docs/guides/publishing-napi-modules/
The text was updated successfully, but these errors were encountered: