nodejs / nodejs/node-api-cts

Drop Node.js v20 from CI matrix

Open Beginner friendly
#37 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
C
Stars
18
Forks
12
PR merge metrics
No merged PRs in 30d

Description

Context

Node.js v20 reaches EOL on April 30, 2026. Additionally, Node-API version 10 is not fully supported on v20 — the NAPI_PREAMBLE macro only returns napi_cannot_run_js for NAPI_VERSION_EXPERIMENTAL, not for version 10+. This was fixed in nodejs/node#55676 ("node-api: define version 10"), which landed in v22.14.0+ but was never backported to v20.

This causes test_cannot_run_js (and potentially other tests targeting Node-API v10) to crash with SIGSEGV on v20, as seen in #28.

Proposal

Remove 20.x from the CI test matrix in .github/workflows/test.yml, along with the Node 20-specific amaro/TypeScript stripping workaround.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Inspect .github/workflows/test.yml, starting with the CI test matrix and the Node 20-specific amaro/TypeScript stripping workaround. Remove the 20.x entry and that workaround; done when the workflow targets the remaining Node.js versions without the v20-specific handling.

Written by the indexing model from the issue text.

Assessment

Tech stack
github-actions, node.js, typescript
Domain
ci-cd
Issue type
Refactor
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Quiet
Clarity
Clearly specified
Newbie friendliness
68/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.