Skip to content

ci: move actions to v5 for the node24 runtime - #1

Merged
viny4 merged 1 commit into
mainfrom
dev
Jul 29, 2026
Merged

ci: move actions to v5 for the node24 runtime#1
viny4 merged 1 commit into
mainfrom
dev

Conversation

@viny4

@viny4 viny4 commented Jul 29, 2026

Copy link
Copy Markdown
Owner

actions/checkout@v4 and setup-node@v4 run on node20, which GitHub is retiring; runners already force them onto node24 and warn. v5 is the first release that targets node24 natively.

Staying on v5 rather than the latest v7 because v5 is the smallest change that clears the warning, and the newer majors alter defaults this workflow does not need.

What does this change?

Why?

How was it tested?

Checklist

  • npm test passes
  • No new runtime dependencies (Electron is the only one)
  • No innerHTML used with user-controlled data
  • Security model unchanged (contextIsolation on, nodeIntegration off)
  • No telemetry, analytics, or third-party requests added
  • Logic with real rules in it lives in lib/ with tests

actions/checkout@v4 and setup-node@v4 run on node20, which GitHub is
retiring; runners already force them onto node24 and warn. v5 is the
first release that targets node24 natively.

Staying on v5 rather than the latest v7 because v5 is the smallest change
that clears the warning, and the newer majors alter defaults this workflow
does not need.
@viny4
viny4 merged commit c7bc959 into main Jul 29, 2026
5 checks passed
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.

1 participant