Skip to content

ci: remove Node.js 18 and 23, add 24 to test workflow#512

Merged
BaseMax merged 1 commit intotcort:masterfrom
MikeMcC399:update/ci-node
Sep 22, 2025
Merged

ci: remove Node.js 18 and 23, add 24 to test workflow#512
BaseMax merged 1 commit intotcort:masterfrom
MikeMcC399:update/ci-node

Conversation

@MikeMcC399
Copy link
Contributor

Situation

The "Test library" workflow .github/workflows/ci.yml is using Node.js versions that have reached end-of-life, and is not testing against the current version of Node.js.

Referring to the Node.js release schedule the following release transitions have occurred:

Node.js Status Date
18.x End-of-Life 2025-04-30
23.x End-of-Life 2025-06-01
24.x Release 2025-05-06

Change

For the "Test library" workflow .github/workflows/ci.yml:

remove

  • Node.js 18
  • Node.js 23

add

  • Node.js 24

@MikeMcC399
Copy link
Contributor Author

Not only is Node.js 18 EOL, it also conflicts with commander@14.0.1 in dependencies which requires node >=20

CI using Node.js 18 shows

$ npm ci
npm warn EBADENGINE Unsupported engine {
npm warn EBADENGINE   package: 'commander@14.0.1',
npm warn EBADENGINE   required: { node: '>=20' },
npm warn EBADENGINE   current: { node: 'v18.20.8', npm: '10.8.2' }
npm warn EBADENGINE }

Node.js 18 should no longer be used.

@BaseMax BaseMax merged commit 17439c9 into tcort:master Sep 22, 2025
9 checks passed
@BaseMax
Copy link
Collaborator

BaseMax commented Sep 22, 2025

Many thanks.

@MikeMcC399 MikeMcC399 deleted the update/ci-node branch September 22, 2025 17:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants