Use Node.js 12 in CI

This commit is contained in:
John J. Camilleri
2021-10-18 14:42:37 +02:00
parent 42c1ec4448
commit 0b13d04ac4
2 changed files with 15 additions and 6 deletions

View File

@@ -84,7 +84,6 @@ jobs:
name: JavaScript (Ubuntu)
runs-on: ubuntu-20.04
needs: ubuntu-runtime
if: false
steps:
- uses: actions/checkout@v2
@@ -96,6 +95,11 @@ jobs:
sudo mv lib/* /usr/local/lib/
sudo mv include/* /usr/local/include/
- name: Setup Node.js
run: |
nvm install 12
nvm use 12
- name: Install dependencies
working-directory: ./src/runtime/javascript
run: |
@@ -193,7 +197,6 @@ jobs:
name: JavaScript (macOS)
runs-on: macOS-11
needs: macos-runtime
if: false
steps:
- uses: actions/checkout@v2
@@ -205,6 +208,11 @@ jobs:
sudo mv lib/* /usr/local/lib/
sudo mv include/* /usr/local/include/
- name: Setup Node.js
run: |
nvm install 12
nvm use 12
- name: Install dependencies
working-directory: ./src/runtime/javascript
run: |

View File

@@ -31,11 +31,12 @@ npm run build
npm run test
```
## Known issues
## Compatability
Unfortunately, these bindings currently only work properly with Node.js < 14.
Newer versions of Node.js case the bindings to crash randomly, and/or run very slowly.
This issue is discussed here:
There is a known issue with random crashes with Node 14 and 16 on macOS. See here:
- https://github.com/nodejs/node/issues/32463
- https://github.com/node-ffi-napi/node-ffi-napi/issues/97
- https://github.com/node-ffi-napi/ref-napi/issues/47
It seems to work with Node 12 on macOS, as well as on other platforms.