Skip to content

feat(ci): test and publish on Node 24 LTS - #248

Merged
ihor-sokoliuk merged 1 commit into
mainfrom
codex/ci/node-24-lts
Aug 18, 2026
Merged

feat(ci): test and publish on Node 24 LTS#248
ihor-sokoliuk merged 1 commit into
mainfrom
codex/ci/node-24-lts

Conversation

@ihor-sokoliuk

Copy link
Copy Markdown
Owner

Summary

  • test the declared Node 20 minimum and maintained Node 24 LTS in CI with independent matrix results
  • run both tag-triggered publishing jobs on Node 24
  • preserve the exact npm 11.17.0 trusted-publishing pin and the Node 20 package floor

Reimplements the useful maintained-runtime scope from #175 against current main; the contributor branch was not modified.

Testing

  • Node 20.20.2 / npm 11.17.0: lint, build, 704/704 tests, 96.10% line and 92.00% branch coverage, 26/26 E2E, packed consumer
  • Node 24.19.0 / npm 11.17.0: lint, build, 704/704 tests, 94.75% line and 91.98% branch coverage, 26/26 E2E, packed consumer
  • exact workflow contract, diff check, conflict scan, and secret preflight

Coverage: 94.75% (was 94.75%)
Copilot AI lite review requested due to automatic review settings August 18, 2026 11:48

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates GitHub Actions workflows to validate the project on both the declared minimum Node.js version (20) and the maintained LTS runtime (24), while ensuring publishing continues to run on Node 24 with the existing trusted-publishing npm pin preserved.

Changes:

  • Add a CI matrix to run lint/build/tests on Node 20 and Node 24 (with independent results).
  • Update both npm publishing jobs to run on Node 24.
  • Adjust workflow commentary to keep the “exact npm pin” rationale accurate regardless of runner-bundled npm versions.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
.github/workflows/ci.yml Adds a Node 20/24 matrix (fail-fast disabled) and uses the matrix version in setup-node.
.github/workflows/npm-publish.yml Switches publish jobs to Node 24 and updates trusted-publishing pin commentary to be runner-version-agnostic.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@ihor-sokoliuk
ihor-sokoliuk merged commit 71d65ab into main Aug 18, 2026
6 checks passed
@ihor-sokoliuk
ihor-sokoliuk deleted the codex/ci/node-24-lts branch August 18, 2026 19:41
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