Proposal to conform to new N-API interface
While this interface is still "experimental," it is being championed by IBM as the strategic future for native add-ons to the language. For API specifications, see https://nodejs.org/api/n-api.html
I suspect @WorksOfLiam, @mengxumx, @aaronbartell, or others could tackle this, so opening the issue so we can have a public discussion on how to best proceed. Perhaps some other folks from the IBM i community would also be interested in helping.
Comments (8)
-
Account Deactivated -
reporter https://github.com/nodejs/node-addon-api/blob/master/tools/conversion.js
This conversion tool is a good place to start on the project. It's expected to get about 80% of the conversion done (and they will take issues for anything that reasonably should have been converted and wasn't)
This article may also be useful: https://medium.com/the-node-js-collection/n-api-next-generation-node-js-apis-for-native-modules-169af5235b06
-
Hi
Sorry for the late reply. If I can find the time I will happily make a PR, until then I will post some notes about what I might do in case anyone else wants to give it a shot.
- Build nodejs-idb-connector on IBM i with Node.js 8.6.0 or above
- Once built, attempt to use
converter.js
which is found in the node-addon-api repository on GitHub. - I hear that this will convert around 80% (don't quote me on that) and then it's manual work from there.
Hopefully this is a step in the right direction.
Cheers, Liam
-
reporter Created branch N-API-compliance to serve as a home for this major rework
-
reporter -
assigned issue to
-
assigned issue to
-
reporter -
assigned issue to
-
assigned issue to
-
Ran it through the conversion tool the output is located: https://bitbucket.org/abmusse/nodejs-idb-connector
Have not built it yet, but getting to it soon.
-
reporter - changed status to resolved
I believe this to be resolved with the latest commits. Thanks!!
- Log in to comment
Issue
#9was marked as a duplicate of this issue.