Skip to content

Commit

Permalink
fix(breaking-change): update for nodes 18 and 20 (#53)
Browse files Browse the repository at this point in the history
  • Loading branch information
shazron authored Nov 23, 2023
1 parent a23494e commit 0fd86c3
Show file tree
Hide file tree
Showing 8 changed files with 12 additions and 12 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/daily.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ jobs:
runs-on: ${{ matrix.os }}
strategy:
matrix:
node-version: [14]
node-version: [18]
os: [ubuntu-latest]

steps:
Expand All @@ -30,5 +30,5 @@ jobs:
env:
SLACK_WEBHOOK: ${{ secrets.SLACK_WEBHOOK }}
SLACK_TITLE: 'Node version'
SLACK_MESSAGE: ${{ matrix.node }}
SLACK_MESSAGE: ${{ matrix.node-version }}
SLACK_COLOR: ${{ job.status == 'success' && 'good' || job.status == 'cancelled' && '#808080' || 'danger' }}
2 changes: 1 addition & 1 deletion .github/workflows/node.js.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ jobs:
runs-on: ${{ matrix.os }}
strategy:
matrix:
node-version: [14.x, 16.x]
node-version: [18.x, 20.x]
os: [ubuntu-latest, windows-latest]

steps:
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/on-push-publish-to-npm.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ jobs:
- uses: actions/checkout@v2
- uses: actions/setup-node@v1
with:
node-version: 14
node-version: 18
- run: npm install
- run: npm test
- uses: JS-DevTools/npm-publish@v1
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/prerelease.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ jobs:
git config user.email [email protected]
- uses: actions/setup-node@v1
with:
node-version: 16
node-version: 18
- run: |
npm install
npm test
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/version-bump-publish.yml
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ jobs:
git config user.email [email protected]
- uses: actions/setup-node@v1
with:
node-version: 14
node-version: 18
- run: |
npm install
npm test
Expand Down
4 changes: 2 additions & 2 deletions lib/constants.js
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,6 @@ module.exports = {
},
appConfigFile: 'app.config.yaml',
runtimeManifestKey: 'runtimeManifest',
nodeEngines: '^14.18 || ^16.13 || >=18',
defaultRuntimeKind: 'nodejs:16'
nodeEngines: '>=18',
defaultRuntimeKind: 'nodejs:18'
}
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
"url": "https://github.com/adobe/generator-app-common-lib.git"
},
"engines": {
"node": "^14.18 || ^16.13 || >=18"
"node": ">=18"
},
"devDependencies": {
"@types/jest": "^29.5.0",
Expand Down
6 changes: 3 additions & 3 deletions test/lib/EventGenerator.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -230,7 +230,7 @@ describe('implementation', () => {
'test-action-name-existing': {
function: 'actions/generic/index.js',
web: 'yes',
runtime: 'nodejs:16',
runtime: 'nodejs:18',
inputs: {
LOG_LEVEL: 'debug'
},
Expand Down Expand Up @@ -290,7 +290,7 @@ describe('implementation', () => {
'test-action-name-prev': {
function: 'actions/generic/index.js',
web: 'yes',
runtime: 'nodejs:16',
runtime: 'nodejs:18',
inputs: {
LOG_LEVEL: 'debug'
},
Expand Down Expand Up @@ -348,7 +348,7 @@ describe('implementation', () => {
'test-action-name-existing': {
function: 'actions/generic/index.js',
web: 'yes',
runtime: 'nodejs:16',
runtime: 'nodejs:18',
inputs: {
LOG_LEVEL: 'debug'
},
Expand Down

0 comments on commit 0fd86c3

Please sign in to comment.