Skip to content

fix(ci): drop Node 18.x from the test matrix (EOL, vitest 4 requires 20.12+)#39

Open
asachs01 wants to merge 1 commit into
mainfrom
fix/drop-node18-eol-matrix
Open

fix(ci): drop Node 18.x from the test matrix (EOL, vitest 4 requires 20.12+)#39
asachs01 wants to merge 1 commit into
mainfrom
fix/drop-node18-eol-matrix

Conversation

@asachs01

Copy link
Copy Markdown
Member

Node 18.x CI job fails at Vitest startup - vitest 4.x imports styleText from node:util, added in Node 20.12+. Not a code bug, a matrix gap: Node 18 is EOL per the orgs 2026-07-18 EOL-Node audit (wyre-technology/.github PR #35). Drops 18.x from the matrix, fixes the stale engines.node (was >=18.0.0, now >=20.0.0).

Verified locally: lint/test/build all clean. Letting CI confirm on 20.x/22.x before merge.

…20.12+)

vitest 4.x imports styleText from node:util, added in Node 20.12 - the
Node 18.x job fails at Vitest startup with a SyntaxError before any
test runs, unrelated to the actual code. Node 18 is EOL per the orgs
2026-07-18 EOL-Node audit (wyre-technology/.github PR 35), so dropping
it from the matrix rather than trying to keep vitest working on it.
Also fixed the stale engines.node (was >=18.0.0, now >=20.0.0) to match
the real runtime requirement.

Verified locally: lint/test/build all clean on Node v26.4.0.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

1 participant