Skip to content

Upgrade GitHub Actions#2910

Open
imsys wants to merge 1 commit intonesbox:mainfrom
imsys:patch-10
Open

Upgrade GitHub Actions#2910
imsys wants to merge 1 commit intonesbox:mainfrom
imsys:patch-10

Conversation

@imsys
Copy link
Contributor

@imsys imsys commented Mar 12, 2026

Trying to fix these warnings:

Node.js 20 actions are deprecated. The following actions are running on Node.js 20 and may not work as expected: actions/checkout@v4, actions/upload-artifact@v4. Actions will be forced to run with Node.js 24 by default starting June 2nd, 2026. Please check if updated versions of these actions are available that support Node.js 24. To opt into Node.js 24 now, set the FORCE_JAVASCRIPT_ACTIONS_TO_NODE24=true environment variable on the runner or in your workflow file. Once Node.js 24 becomes the default, you can temporarily opt out by setting ACTIONS_ALLOW_USE_UNSECURE_NODE_VERSION=true. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/

Screenshot from 2026-03-12 20-28-58

@imsys imsys marked this pull request as ready for review March 12, 2026 23:49
@imsys
Copy link
Contributor Author

imsys commented Mar 13, 2026

Still getting the warning for: windows-mingw and html at https://github.com/nesbox/TIC-80/actions/runs/23028853744?pr=2910

For the html task, it depends on mymindstorm/setup-emsdk#50

For the mingw, it uses msys2/setup-msys or ruby/setup-ruby. Ruby is used in other builds that are not showing the warning, so the problem seems to be from msys2/setup-msys, but I can't say for sure.

I'd say to approve this first, and later we try to fix the missing ones.

@RobLoach
Copy link
Contributor

Emscripten is knowing slow to update to the latest Node.js. If you find their sources, you'll see many issues related to it:
https://github.com/emscripten-core/emsdk/issues?q=is%3Aissue%20state%3Aopen%20node

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