-
Notifications
You must be signed in to change notification settings - Fork 31
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
Reimplementation in N-API #33
Labels
Comments
Thank you for taking initiative to create this ticket. I know all of us are busy. Just wanted to check if there is any progress on this since node v10 is EOL and is not being supported anymore |
Merged
@cheery , when are you going to release the package in npm? |
I'm going to release npm later this week.
ti 27. heinäk. 2021 klo 14.43 Gaikwad Pratik ***@***.***>
kirjoitti:
… @cheery <https://github.com/cheery> ,
when are you going to release the package in nom?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#33 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAJIXPEODOCJXL5QKPQXKU3TZ2LXLANCNFSM4RE4DDPA>
.
|
any idea when would you be releasing it in nom? |
I did run npm publish during the weekend. Anything elsewhere that is needed?
ti 10. elok. 2021 klo 18.03 Gaikwad Pratik ***@***.***>
kirjoitti:
… @cheery <https://github.com/cheery>,
any idea when would you be releasing it in nom?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#33 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAJIXPFXIBJ7XGWFMQ7K53TT4E5VVANCNFSM4RE4DDPA>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email>
.
|
nope. Thanks!!! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
As a response to discussion on the issue #32, we examine the possibility to reimplement the library with N-API. The motivation to do this change is that this interface is purported to be ABI-stable.
The secondary objective here is to also provide WebIDL specifications and TypeScript headers for the library, to allow easier interop from numerous languages that compile to Js.
This change is allowed to be backwards-breaking, and we will bump up the version number by major. Still the library users are considered. We will document why the interface changes, for each function. We also attempt to keep the new library useful for the same things the previous library was useful for.
Resources for the project:
A new branch (v1) was created for the work toward the N-API bindings. Help is welcome and we will work on this together.
The text was updated successfully, but these errors were encountered: