Skip to content

Commit

Permalink
chore!: Drop node 14 support (grain-lang#1092)
Browse files Browse the repository at this point in the history
chore(cli): Remove `--experimental-wasm-bigint` flag
  • Loading branch information
spotandjake authored May 15, 2022
1 parent 51170e7 commit ef4358f
Show file tree
Hide file tree
Showing 7 changed files with 9 additions and 15 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ jobs:
fail-fast: false
matrix:
os: [ubuntu-latest, macos-latest, windows-latest]
node-version: ["14", "16"]
node-version: ["16"]

steps:
- name: Setup Node.js
Expand Down
6 changes: 3 additions & 3 deletions .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ jobs:
- name: Setup Node.js
uses: actions/setup-node@v2.1.2
with:
node-version: "14"
node-version: "16"
check-latest: true

- name: Setup environment
Expand Down Expand Up @@ -197,7 +197,7 @@ jobs:
- name: Setup NodeJS
uses: actions/setup-node@v2.1.2
with:
node-version: "14"
node-version: "16"
registry-url: "https://registry.npmjs.org"

- name: Publish to npm
Expand All @@ -215,7 +215,7 @@ jobs:
- name: Setup NodeJS
uses: actions/setup-node@v2.1.2
with:
node-version: "14"
node-version: "16"
registry-url: "https://registry.npmjs.org"

- name: Publish to npm
Expand Down
8 changes: 1 addition & 7 deletions cli/bin/grain.js
Original file line number Diff line number Diff line change
Expand Up @@ -8,14 +8,8 @@ const v8 = require("v8");
* This method should be used with care. Changing settings after the VM has started may result
* in unpredictable behavior, including crashes and data loss; or it may simply do nothing.
*
* This seems to work for our needs with Node 14, but we should be cautious when updating.
* This seems to work for our needs with Node 16, but we should be cautious when updating.
*/
if (
process.versions.node.startsWith("14.") ||
process.versions.node.startsWith("15.")
) {
v8.setFlagsFromString("--experimental-wasm-bigint");
}
v8.setFlagsFromString("--experimental-wasm-return-call");

const program = require("commander");
Expand Down
2 changes: 1 addition & 1 deletion cli/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
"description": "A command line tool for the Grain language.",
"main": "index.js",
"engines": {
"node": ">=14"
"node": ">=16"
},
"scripts": {
"clean": "del-cli 'bin/*.exe' 'bin/*.bc.js'",
Expand Down
2 changes: 1 addition & 1 deletion compiler/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
"grainc": "_esy/default/build/install/default/bin/grainc"
},
"engines": {
"node": ">=14"
"node": ">=16"
},
"devDependencies": {
"esy": "0.6.12"
Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
"compiler"
],
"engines": {
"node": ">=14"
"node": ">=16"
},
"scripts": {
"prepare": "yarn js-runner build && yarn cli link",
Expand Down
2 changes: 1 addition & 1 deletion stdlib/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@
"directory": "stdlib"
},
"engines": {
"node": ">=14"
"node": ">=16"
},
"main": "index.js",
"files": [
Expand Down

0 comments on commit ef4358f

Please sign in to comment.