Add Node.js 24 to test strategy#71091
Conversation
When both `node-version-file` and `node-version` are passed to `actions/setup-node`, a warning is output. When inputs are empty, they are ignored. This will silence the error while keepign the same functionality.
This reverts commit 8f95ec0.
|
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message. To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
aduth
left a comment
There was a problem hiding this comment.
Makes sense to me assuming they already pass as-is 👍
sirreal
left a comment
There was a problem hiding this comment.
Good to get ahead of this, thanks!
|
The v24 LTS is now available. What are the next steps for making this the "default" version? Following your earlier comment and specifically around having a "compelling argument", one relatively significant feature I'm hoping to start leveraging is native TypeScript support. For example, a lot of the new build tooling that @youknowriad has been championing through #72032 would benefit from this (example). |
|
Another compelling feature that is only in 24 and late 22 versions is the findPackageJSON function in the |
|
Created an issue for tracking LTS update in #72973 |
What?
This adds version 24 of Node.js to the testing strategies for workflows that test multiple versions of Node.js.
Why?
Testing against newer versions than the one currently pinned in the WordPress Core/Gutenberg repositories helps spot potential issues ahead of time making upgrading easier when the time comes.
See related: #70844.
How?
The GitHub Actions testing strategies have been updated for workflows that currently test multiple versions of Node.js