PR-5.4-dev Pending

User tests: Successful: Unsuccessful:

avatar tecpromotion
tecpromotion
24 Sep 2025

Summary of Changes

Update actions/checkout from v3 to v5
Update actions/setup-node from v3 to v5
Update node version from 20 to 24

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

avatar tecpromotion tecpromotion - open - 24 Sep 2025
avatar tecpromotion tecpromotion - change - 24 Sep 2025
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 24 Sep 2025
Category Repository
avatar C-Lodder
C-Lodder - comment - 25 Sep 2025

Current Node.js LTS is v22.

avatar richard67
richard67 - comment - 25 Sep 2025

@tecpromotion Could you change it to node v22? See previous comment. Thanks in advance.

avatar laoneo
laoneo - comment - 25 Sep 2025

Honestly it doesn't matter. As long as it is actual, I would leave it on version 24.

avatar C-Lodder
C-Lodder - comment - 25 Sep 2025

@laoneo Not for releases, but could cause problems for local development.

For example, if someone submits a PR for the build scripts, which uses RegExp.escape(...), the tests will pass and the PR will most likely be merged, however if any developers are using Node.js LTS (v22 currently), this code won't work at all, because it was included in V8 13.6, which was shipped with Node.js 24.

It's a major version change, which includes features unsupported in v22.

Just my 2 cents

avatar laoneo
laoneo - comment - 25 Sep 2025

Ok. I don't think so running the translation workflows on a newer version of node is a problem.

avatar richard67
richard67 - comment - 25 Sep 2025

Wouldn't using an STS mean we have to change the version again sooner than it would be the case with an LTS, if we don't want to run with a version which has reached end of support?

I know, it's only the GitHub actions.

But everywhere else we use (or require) LTS.

avatar laoneo
laoneo - comment - 25 Sep 2025

I think GH actions don't fall under the minimum requirements. But it is fine to change to LTS as well. Just wanted to point out that here it should be fine not using LTS. I see a benefit of using different versions, like that we know it works also on the latest.

avatar tecpromotion
tecpromotion - comment - 25 Sep 2025

We use this GHA ONLY and EXCLUSIVELY in the translation-bot repo! This change has no effect here or in local environments. Or am I overlooking something @HLeithner?
Also see: Breaking changes in V5

avatar HLeithner
HLeithner - comment - 25 Sep 2025

This has nothing to do with joomla or it's build process, and can be merged. No idea why it's not merged already.

avatar richard67 richard67 - change - 25 Sep 2025
Labels Added: PR-5.4-dev
avatar richard67
richard67 - comment - 25 Sep 2025

No idea why it's not merged already.

@HLeithner Because you haven't approved it yet? 😛

avatar richard67 richard67 - change - 25 Sep 2025
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2025-09-25 14:14:19
Closed_By richard67
avatar richard67 richard67 - close - 25 Sep 2025
avatar richard67 richard67 - merge - 25 Sep 2025
avatar richard67
richard67 - comment - 25 Sep 2025

Thanks all (author and reviewers).

avatar laoneo
laoneo - comment - 25 Sep 2025

This has nothing to do with joomla or it's build process, and can be merged. No idea why it's not merged already.

It does run the build script here in node 24 with this pr. So @C-Lodder is right when it is vice versa. It might break, when the build script has a feature, which was removed in 24.

avatar richard67
richard67 - comment - 25 Sep 2025

Then someone should make a PR to change to v22.

avatar richard67
richard67 - comment - 25 Sep 2025

We definitely have to go to v22 as for a new build of a new PR we now get the same error as @bembelimen recently had when building with node v24:

file:///__w/joomla-cms/joomla-cms/build/build-modules-js/stylesheets/css-versioning.mjs:83
    throw new Error(error);
          ^
Error: Error: ENOENT: no such file or directory, open 'undefined/content.css'
    at fixVersion (file:///__w/joomla-cms/joomla-cms/build/build-modules-js/stylesheets/css-versioning.mjs:83:11)
    at async Promise.all (index 0)

The error is not related to the recent update of css-versioning.mjs, it happened before that when using node v24.

I will make a PR to change to v22 in a few minutes.

avatar richard67
richard67 - comment - 25 Sep 2025

@laoneo @HLeithner Please check and approve PR #46169 . Thanks in advance.

avatar richard67
richard67 - comment - 25 Sep 2025

Hmm, I do not see the relationship now as this PR only changed it for the translation PR actions.

However, the CI action use node version = latest.

And so that runs with v24 now, too.

Add a Comment

Login with GitHub to post a comment