Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[4.4] Node 20 #42331

Merged
merged 3 commits into from
Nov 11, 2023
Merged

[4.4] Node 20 #42331

merged 3 commits into from
Nov 11, 2023

Conversation

dgrammatiko
Copy link
Contributor

Pull Request for Issue #42265 (back porting to 4.4) .

Summary of Changes

  • Node 18 Active Support ended 2 weeks ago (18 Oct 2023), so switching to 20
  • Added missing semver package in the devDependencies
  • use the helper handleErr fn
  • Added a runtime conditional and a message for incompatible node versions

Testing Instructions

Run npm ci, everything still works

Actual result BEFORE applying this Pull Request

Expected result AFTER applying this Pull Request

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

Signed-off-by: Dimitris Grammatikogiannis <dg@dgrammatiko.dev>
@joomla-cms-bot joomla-cms-bot added Language Change This is for Translators NPM Resource Changed This Pull Request can't be tested by Patchtester PR-5.0-dev Unit/System Tests labels Nov 11, 2023
@dgrammatiko dgrammatiko changed the base branch from 5.0-dev to 4.4-dev November 11, 2023 10:14
@joomla-cms-bot joomla-cms-bot added PR-4.4-dev and removed Language Change This is for Translators labels Nov 11, 2023
@richard67
Copy link
Member

This PR fixes this issue: #42325 (comment)

I could reproduce it locally on an Ubuntu VM and have verified that it's fixed on the branch of this PR.

For fixing the currently failing 4.4-dev nightly build it will need this PR here and in addition joomla-projects/docker-images#72 .

@laoneo laoneo added this to the Joomla! 4.4.1 milestone Nov 11, 2023
@laoneo laoneo merged commit 584d7b0 into joomla:4.4-dev Nov 11, 2023
3 checks passed
@laoneo
Copy link
Member

laoneo commented Nov 11, 2023

Thanks all for the quick fix.

@dgrammatiko dgrammatiko deleted the 4.4-dev-tools branch November 11, 2023 15:41
@dgrammatiko
Copy link
Contributor Author

@laoneo please also commit joomla-projects/docker-images#72 to fix the packager. Right now it's (probably) still broken:
Screenshot 2023-11-11 at 16 45 17

@heelc29 heelc29 mentioned this pull request Oct 19, 2024
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
NPM Resource Changed This Pull Request can't be tested by Patchtester PR-4.4-dev Unit/System Tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants