Feature ? Conflicting Files NPM Resource Changed PR-5.0-dev Pending

User tests: Successful: Unsuccessful:

avatar dgrammatiko
dgrammatiko
1 Nov 2023

Pull Request for Issue # .

Summary of Changes

  • Node 18 Active Support ended 2 weeks ago (18 Oct 2023), so switching to 20
  • Added .npmrc so the node version is strict
  • 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

avatar dgrammatiko dgrammatiko - open - 1 Nov 2023
avatar dgrammatiko dgrammatiko - change - 1 Nov 2023
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 1 Nov 2023
Category Unit Tests Repository JavaScript NPM Change
avatar dgrammatiko dgrammatiko - change - 1 Nov 2023
The description was changed
avatar dgrammatiko dgrammatiko - edited - 1 Nov 2023
avatar dgrammatiko dgrammatiko - change - 1 Nov 2023
The description was changed
avatar dgrammatiko dgrammatiko - edited - 1 Nov 2023
avatar dgrammatiko dgrammatiko - change - 1 Nov 2023
Labels Added: ? NPM Resource Changed PR-5.0-dev
0f88c10 2 Nov 2023 avatar dgrammatiko gt
avatar dgrammatiko dgrammatiko - change - 2 Nov 2023
Labels Added: ?
Removed: ?
avatar dgrammatiko dgrammatiko - change - 2 Nov 2023
The description was changed
avatar dgrammatiko dgrammatiko - edited - 2 Nov 2023
18e4153 2 Nov 2023 avatar dgrammatiko CS
avatar bembelimen
bembelimen - comment - 15 Nov 2023

Hey @dgrammatiko would you please check the upmerge #42345 if this resolved this PR or if something is missing? Thanks!

avatar dgrammatiko dgrammatiko - close - 15 Nov 2023
avatar dgrammatiko dgrammatiko - change - 15 Nov 2023
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2023-11-15 17:47:06
Closed_By dgrammatiko
Labels Added: Feature ? Conflicting Files
Removed: ?
avatar dgrammatiko
dgrammatiko - comment - 15 Nov 2023

This can be closed although there are some mistakes that need fixing. Will do a pr once I get some free time

Add a Comment

Login with GitHub to post a comment