Skip to content

Commit

Permalink
Upgrade to mocha v11, resolve CVE warning, update minimum to Node.j…
Browse files Browse the repository at this point in the history
…s 18.x, add 22.x to CI
  • Loading branch information
ryanblock committed Jan 28, 2025
1 parent 40ad5ff commit 5bf01f6
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 9 deletions.
12 changes: 6 additions & 6 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,16 +15,16 @@ jobs:
runs-on: ${{ matrix.os }}
strategy:
matrix:
node-version: [ 14.x, 16.x, 18.x ]
node-version: [ 18.x, 20.x, 22.x ]
os: [ windows-latest, ubuntu-latest, macOS-latest ]

# Go
steps:
- name: Check out repo
uses: actions/checkout@v3
uses: actions/checkout@v4

- name: Set up Node.js
uses: actions/setup-node@v3
uses: actions/setup-node@v4
with:
node-version: ${{ matrix.node-version }}

Expand All @@ -48,7 +48,7 @@ jobs:
- name: Notify
uses: sarisia/actions-status-discord@v1
# Only fire alert once
if: github.ref == 'refs/heads/main' && failure() && matrix.node-version == '14.x' && matrix.os == 'ubuntu-latest'
if: github.ref == 'refs/heads/main' && failure() && matrix.node-version == '22.x' && matrix.os == 'ubuntu-latest'
with:
webhook: ${{ secrets.DISCORD_WEBHOOK }}
title: "build and test"
Expand All @@ -67,10 +67,10 @@ jobs:
# Go
steps:
- name: Check out repo
uses: actions/checkout@v3
uses: actions/checkout@v4

- name: Set up Node.js
uses: actions/setup-node@v3
uses: actions/setup-node@v4
with:
node-version: lts/*
registry-url: https://registry.npmjs.org/
Expand Down
2 changes: 1 addition & 1 deletion changelog.md
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ Hello! After a bit of a hiatus, the [Architect team](https://github.com/architec

### Changed

- [Breaking change] Introduced minimum Node.js version of >= 14; fixes [#169](https://github.com/architect/dynalite/issues/169)
- [Breaking change] Introduced minimum Node.js version of >= 18; fixes [#169](https://github.com/architect/dynalite/issues/169)
- [Breaking change] When using SSL mode, you must now supply your own `key`, `cert`, and `ca` (which isn't much of a breaking change, really, because Dynalite's certs were expired); fixes [#176](https://github.com/architect/dynalite/issues/176)
- In CLI mode, pass them as file paths with flags (e.g. `--key /foo/key.pem --cert /foo/cert.pem --ca /foo/ca-cert.pem`)
- As a module, you can pass them as strings or as file paths; when passing as file paths, make sure you include a boolean `useSSLFilePaths` option
Expand Down
4 changes: 2 additions & 2 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@
"lint": "eslint . --fix"
},
"engines": {
"node": ">=14"
"node": ">=18"
},
"author": "Michael Hart <[email protected]>",
"license": "Apache-2.0",
Expand All @@ -40,7 +40,7 @@
"@architect/eslint-config": "^2.1.1",
"aws4": "^1.12.0",
"eslint": "^8.48.0",
"mocha": "^10.2.0",
"mocha": "^11.1.0",
"pegjs": "^0.10.0",
"should": "^13.2.3"
},
Expand Down

0 comments on commit 5bf01f6

Please sign in to comment.