-
Notifications
You must be signed in to change notification settings - Fork 16
Migrate from NAN -> NAPI #281
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
base: master
Are you sure you want to change the base?
Conversation
Hey @cowboyd, thanks for the PR! |
@milindl Thanks! I'm sure that it's broken in a number of ways, but my goal was to get most of the grunt work out of the way. I'd be happy to help you all push further with this as I think it would be killer to get this working on Deno and Bun. Let me know! |
What
N-API is a forward ABI compatible API that is supported by Node, Deno, and Bun. By using N-API instead of NAN,
@confluent/kafka-javascript
will be able to run on all those runtimes as well as any other runtime that comes along that supports N-API.Checklist
References
JIRA:
Test & Review
Open questions / Follow-ups