search

nodejs / abi-stable-node

Repository used by the Node-API team to manage work related to Node-API and node-addon-api
239 stars 47 forks source link

N-API 7 #402

Closed gabrielschulhof closed 3 years ago

gabrielschulhof commented 4 years ago

Might it be time for a new version of N-API?

We have the buffer detaching stuff marked as experimental in all three LTS releases, so we could just define these APIs as N-API 7.

We also have type-tagging and asynchronous env cleanup coming on master, but those still need to be backported. As v10.x approaches EOL it is increasingly unlikely that they will be backported all the way there. Thus, it might make sense to sync our N-API version bump with the EOL of 10.x.

Shall we formalize and document our policy for moving APIs out of experimental and into a stable version of N-API?

mhdawson commented 4 years ago

We agreed:

gabrielschulhof commented 3 years ago

@Flarna asked that https://github.com/nodejs/node/pull/35214 be folded into N-API 7 because it's such a small change. Let's discuss at our next meeting.

mhdawson commented 3 years ago

All ready to go, just waiting next 10.x release and then we can close this.

mhdawson commented 3 years ago

Landed in 10.x, done !