Skip to content

ci: enforce Node 24 across workflows and engines - #20

Merged
antonin-morel-spiko merged 1 commit into
mainfrom
ci/enforce-node-24
Oct 2, 2026
Merged

antonin-morel-spiko merged 1 commit into
mainfrom
ci/enforce-node-24

Conversation

@antonin-morel-spiko

Copy link
Copy Markdown
Collaborator

npm trusted publishing (OIDC) requires npm >= 11.5.1. Node 20 ships npm 10, which published without auth and failed with E404.

npm trusted publishing (OIDC) requires npm >= 11.5.1. Node 20 ships npm 10,
which published without auth and failed with E404.
@antonin-morel-spiko antonin-morel-spiko self-assigned this Oct 2, 2026
Copilot AI balanced review requested due to automatic review settings October 2, 2026 08:49

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.

Copilot review overview

🟢 Approval recommended

The dependency metadata and all workflows consistently enforce Node.js 24 without identified issues.

Review effort: Balanced
Findings: None

What changed in this PR

Aligns development, CI, and publishing with Node.js 24 to support npm trusted publishing.

Changes:

  • Requires Node.js 24 and updates Node type definitions.
  • Standardizes all workflows on Node.js 24.
  • Regenerates dependency lock data.
File Description
package.json Updates Node engine and type definitions.
bun.lock Locks updated Node-related types.
.github/​workflows/​main.yml Runs CI on Node.js 24.
.github/​workflows/​release.yml Uses Node.js 24 for publishing.
.github/​workflows/​size.yml Adds Node.js 24 setup.

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

@antonin-morel-spiko
antonin-morel-spiko merged commit ea1c854 into main Oct 2, 2026
5 checks passed
@antonin-morel-spiko
antonin-morel-spiko deleted the ci/enforce-node-24 branch October 2, 2026 08:52
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