From 5bf01f6dd8c2c80a768c1339bd9f060e9e62148f Mon Sep 17 00:00:00 2001 From: Ryan Block Date: Tue, 28 Jan 2025 11:01:18 -0800 Subject: [PATCH] Upgrade to `mocha` v11, resolve CVE warning, update minimum to Node.js 18.x, add 22.x to CI --- .github/workflows/build.yml | 12 ++++++------ changelog.md | 2 +- package.json | 4 ++-- 3 files changed, 9 insertions(+), 9 deletions(-) diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index 9716bad..e3adfa1 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -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 }} @@ -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" @@ -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/ diff --git a/changelog.md b/changelog.md index c185e21..62b4358 100644 --- a/changelog.md +++ b/changelog.md @@ -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 diff --git a/package.json b/package.json index 3851b1d..99c8873 100644 --- a/package.json +++ b/package.json @@ -17,7 +17,7 @@ "lint": "eslint . --fix" }, "engines": { - "node": ">=14" + "node": ">=18" }, "author": "Michael Hart ", "license": "Apache-2.0", @@ -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" },