Node 16 (as-pect#341)

* experimental-wasm-bigint does not need for node 16 and above * works? * works? * works? * works? * update * more * only test on node 16, but support 15 * fix: revert to use matrix * fix: revert latest scripts * Apply suggestions from code review Co-authored-by: jtenner <tenner.joshua@gmail.com> * fix: formatting Co-authored-by: MaxGraey <maxgraey@gmail.com> Co-authored-by: Willem Wyndham <willem@cs.umd.edu>
jtenner · Apr 27, 2021 · 044ca3a · 044ca3a
1 parent ace6f44
commit 044ca3a
Show file tree

Hide file tree

Showing 14 changed files with 1,784 additions and 1,151 deletions.
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -10,7 +10,7 @@ jobs:
     name: Check syntax
     strategy:
       matrix:
-        node_version: ["15"]
+        node_version: ["16"]
     steps:
     - uses: actions/checkout@v1.0.0
     - uses: dcodeIO/setup-node-nvm@master
@@ -26,7 +26,7 @@ jobs:
     strategy:
       matrix:
         os: [ubuntu-latest, macos-latest, windows-latest]
-        node_version: ["current", "lts_latest", "15"]
+        node_version: ["16"]
     steps:
     - uses: actions/checkout@v1.0.0
     - uses: dcodeIO/setup-node-nvm@master

diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -50,6 +50,5 @@
     "ts-jest": "^26.5.1",
     "ts-node": "^9.1.1",
     "typescript": "^4.1.3"
-  },
-  "dependencies": {}
+  }
 }
diff --git a/packages/cli/bin/asp b/packages/cli/bin/asp
@@ -9,7 +9,7 @@ const args = [
   ...process.argv.slice(2),
 ]
 
-if (nodeVersion >= 12) args.unshift("--experimental-wasm-bigint");
+if (nodeVersion >= 12 && nodeVersion < 16) args.unshift("--experimental-wasm-bigint");
 if (nodeVersion >= 13) args.unshift("--experimental-wasi-unstable-preview1");
 
 const spawned = spawn("node", args, {

diff --git a/packages/cli/package-lock.json b/packages/cli/package-lock.json