Skip to content

fix: update action runtime from Node.js 20 to Node.js 24#827

Merged
frostebite merged 2 commits into
game-ci:mainfrom
ChristianTellefsenAttensi:fix/update-node20-to-node24
May 6, 2026
Merged

fix: update action runtime from Node.js 20 to Node.js 24#827
frostebite merged 2 commits into
game-ci:mainfrom
ChristianTellefsenAttensi:fix/update-node20-to-node24

Conversation

@ChristianTellefsenAttensi

@ChristianTellefsenAttensi ChristianTellefsenAttensi commented Apr 9, 2026

Copy link
Copy Markdown
Contributor

Changes

  • Update action.yml from using: 'node20' to using: 'node24'

Related Issues

Successful Workflow Run Link

  • Tested on a private repository with FORCE_JAVASCRIPT_ACTIONS_TO_NODE24=true set as a top-level env variable. All jobs passed: unit tests, builds for Windows/iOS/Android. We cannot link directly as the repo is private, but happy to share logs or screenshots if needed.

Checklist

  • Read the contribution guide and accept the code of conduct
  • Docs (not needed - no new inputs/outputs)
  • Readme (not needed)
  • Tests (not needed - runtime change only, verified on private repo)

Summary by CodeRabbit

  • Chores
    • Updated the CI action runtime to a newer Node.js environment for improved performance and compatibility.

GitHub is deprecating Node.js 20 on Actions runners:
- June 2, 2026: Node.js 24 forced as default
- September 16, 2026: Node.js 20 removed

Ref: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
Closes game-ci#826
@github-actions

github-actions Bot commented Apr 9, 2026

Copy link
Copy Markdown

Cat Gif

@coderabbitai

coderabbitai Bot commented Apr 9, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: be1dc291-bdf1-491a-baf7-881be524df87

📥 Commits

Reviewing files that changed from the base of the PR and between 49571e8 and 2a8db82.

📒 Files selected for processing (1)
  • action.yml

📝 Walkthrough

Walkthrough

The GitHub Action runtime in action.yml was changed from node20 to node24. The action entry point (dist/index.js) and other configuration remain unchanged.

Changes

Node.js Runtime Update

Layer / File(s) Summary
Manifest change
action.yml
runs.using value updated from node20 to node24 (single-line change).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related issues

Poem

🐰 From twenty to twenty-four I leap,
New runtime wakes from quiet sleep,
A carrot of versions, shiny and bright,
I hop the bytes into the light,
Puff of joy — the action runs right!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main change: updating the GitHub Action runtime from Node.js 20 to Node.js 24, which is precisely what the changeset accomplishes.
Description check ✅ Passed The description follows the template structure with all required sections completed: Changes, Related Issues, Successful Workflow Run Link (with explanation for private repo), and Checklist items properly addressed.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@codecov

codecov Bot commented Apr 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 42.70%. Comparing base (16c5c20) to head (2a8db82).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #827   +/-   ##
=======================================
  Coverage   42.70%   42.70%           
=======================================
  Files          37       37           
  Lines         733      733           
  Branches      201      201           
=======================================
  Hits          313      313           
  Misses        381      381           
  Partials       39       39           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@frostebite

Copy link
Copy Markdown
Member

Branch updated with main. Will merge once CI passes. Thanks @ChristianTellefsenAttensi — deadline is June 2 so this is timely.

@frostebite frostebite merged commit 2240bed into game-ci:main May 6, 2026
7 checks passed
@kraenhansen

Copy link
Copy Markdown

@frostebite any chance we could get things moving over in game-ci/unity-test-runner#304 as well?

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.

3 participants