Skip to content

Declare action runtime as node24 - #315

Merged
mdarocha merged 1 commit into
mainfrom
claude/node24-runtime
Jul 23, 2026
Merged

Declare action runtime as node24#315
mdarocha merged 1 commit into
mainfrom
claude/node24-runtime

Conversation

@mdarocha

Copy link
Copy Markdown
Owner

Summary

GitHub Actions runners already force this action onto Node.js 24 with a deprecation warning, since Node 20 is deprecated:

##[warning]Node.js 20 is deprecated. The following actions target Node.js 20 but are being forced to run on Node.js 24: mdarocha/comment-flake-lock-changelog@...

Observed while smoke-testing nix-magic-setup#23 (which had this action's main branch temporarily pinned) against a real deploy in mdarocha/pondinfra#141.

Changes

  • action.yml: using: node20using: node24.

No other change needed — the build targets a generic node runtime (bun build --target=node) and tsconfig.json targets ESNext, neither pinned to a specific Node version.

🤖 Generated with Claude Code


Generated by Claude Code

GitHub Actions runners already force this action onto Node.js 24 with a
deprecation warning, since Node 20 is deprecated - observed in
nix-magic-setup#23's smoke test (mdarocha/pondinfra#141) when it was
temporarily pinned to main. Declaring node24 removes the warning; no
other change needed since the build targets a generic `node` runtime and
tsconfig targets ESNext, neither pinned to a specific Node version.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_016MYz49D9GAahQN6haYW3Nb
@mdarocha
mdarocha merged commit cc40344 into main Jul 23, 2026
1 check passed
mdarocha pushed a commit to mdarocha/nix-magic-setup that referenced this pull request Jul 23, 2026
Picks up the node24 runtime fix (mdarocha/comment-flake-lock-changelog#315,
merged) so the temporary main pin used for testing change-sentinel no
longer triggers the Node.js 20 deprecation warning.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_016MYz49D9GAahQN6haYW3Nb
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