Skip to content

Commit

Permalink
chore: Bump node version used on GitHub
Browse files Browse the repository at this point in the history
- While [adding trivial change the the code](#365) I bumped into CI being red from various reasons [last one](https://github.com/rwjblue/ember-cli-content-security-policy/actions/runs/8820844298/job/24215355798?pr=365) ended with a hard error on:
```
 [3/5] Fetching packages...
error [email protected]: The engine "node" is incompatible with this module. Expected version "16.* || 18.* || >= 20". Got "12.22.12"
error Found incompatible module.
```
- I tried to reproduce locally, but `[email protected]` is surprisingly not trivial to install on modern OSX without breaking my other projects.
- I tried to consult the [node support](https://github.com/ember-cli/ember-cli/blob/master/docs/node-support.md) matrix for EmberJS and since `16.x` is supported through `3.28.0 - 5.3.0`, it seems like a safe bet.

- After consultation in the PR, a change has been made to support only node 18.x
  • Loading branch information
MichalBryxi committed Apr 25, 2024
1 parent b6c3e09 commit 8c10dcd
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ on:
pull_request:

env:
NODE_VERSION: '12.x'
NODE_VERSION: '18.x'

jobs:
lint:
Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -78,7 +78,7 @@
"semver": "^7.3.5"
},
"engines": {
"node": "12.* || 14.* || >= 16"
"node": ">= 18"
},
"publishConfig": {
"registry": "https://registry.npmjs.org"
Expand Down

0 comments on commit 8c10dcd

Please sign in to comment.