Skip to content

Commit

Permalink
Merge pull request #548 from jelhan/require-node-18
Browse files Browse the repository at this point in the history
drop support for node < 18
  • Loading branch information
knownasilya authored Nov 26, 2024
2 parents 33e31a3 + 2773b55 commit 8a3a71c
Show file tree
Hide file tree
Showing 5 changed files with 4,450 additions and 4,808 deletions.
15 changes: 7 additions & 8 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ on:
pull_request:

env:
NODE_VERSION: '14'
NODE_VERSION: '18'

jobs:
lint:
Expand Down Expand Up @@ -169,16 +169,15 @@ jobs:
strategy:
fail-fast: true
matrix:
ember-try-scenario:
[
ember-try-scenario: [
ember-lts-3.20,
ember-lts-3.24,
ember-release,
ember-beta,
ember-canary,
# ember-release,
# ember-beta,
# ember-canary,
ember-classic,
embroider-safe,
embroider-optimized,
# embroider-safe,
# embroider-optimized,
]

steps:
Expand Down
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -124,7 +124,7 @@ ember install ember-metrics

- Ember.js v3.20 or above
- Ember CLI v3.20 or above
- Node.js v12 or above
- Node.js v18 or above

## Configuration

Expand Down
2 changes: 2 additions & 0 deletions config/ember-try.js
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ module.exports = async function () {
name: 'ember-lts-3.20',
npm: {
devDependencies: {
'ember-qunit': '6.0.0',
'ember-source': '~3.20.5',
},
},
Expand All @@ -19,6 +20,7 @@ module.exports = async function () {
name: 'ember-lts-3.24',
npm: {
devDependencies: {
'ember-qunit': '6.0.0',
'ember-source': '~3.24.3',
},
},
Expand Down
5 changes: 4 additions & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -118,8 +118,11 @@
"typescript": "^4.4.3",
"webpack": "5"
},
"resolutions": {
"@babel/plugin-transform-modules-amd": "7.16.5"
},
"engines": {
"node": "12.* || 14.* || >= 16"
"node": "18.* || 20.* || >= 22"
},
"publishConfig": {
"registry": "https://registry.npmjs.org"
Expand Down
Loading

0 comments on commit 8a3a71c

Please sign in to comment.