all: update tests and workflows

nodejs / webidl-napi

A WebIDL-to-N-API compiler

MIT License

10 stars 6 forks source link

all: update tests and workflows #8

Closed gabrielschulhof closed 4 years ago

gabrielschulhof commented 4 years ago

Add switch to converter to accept custom N-API header.
Rename the GitHub workflows to distinguish them by OS.
Add trigger for pull request.
Propagate individual test build failure through the builder.
Propagate individual test failure through the runner.
Detect Node.js version and use appropriate N-API header in tests' binding.gyp.

Signed-off-by: @gabrielschulhof Fixes: https://github.com/nodejs/webidl-napi/issues/4

cerisano commented 4 years ago

Looks like the CI is using node 10. min req is node 12.

gabrielschulhof commented 4 years ago

@cerisano I wanna make it work on Node.js 10 as well. I'll do a conditional include of js_native_api.h.

gabrielschulhof commented 4 years ago

@kainino0x @austinEng @cerisano sorry for the long wait! I had to switch the build system from node-gyp to cmake-js to get it to properly handle the code generation for the tests on Windows, Linux, and OSX.

gabrielschulhof commented 4 years ago

Landed in df980a4ec6f70d687c8ccd3fef520cdaca20d47f.