Composer Dependency Changed NPM Resource Changed PR-5.3-dev Pending

User tests: Successful: 0 Unsuccessful: 0

avatar laoneo
laoneo
3 Feb 2025

Updates the dependencies.

avatar laoneo laoneo - open - 3 Feb 2025
avatar laoneo laoneo - change - 3 Feb 2025
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 3 Feb 2025
Category External Library Composer Change NPM Change Front End Plugins
avatar richard67
richard67 - comment - 3 Feb 2025

@laoneo It seems the php-cs-fixer update causes some PHPCS error in Drone, very likely due to a new rule or a bug fix in the fixer: https://ci.joomla.org/joomla/joomla-cms/81812/1/7

   1) build/update_fido_cache.php (blank_line_after_opening_tag)
      ---------- begin diff ----------
--- /drone/src/build/update_fido_cache.php
+++ /drone/src/build/update_fido_cache.php
@@ -1,4 +1,5 @@
 <?php
+
 /**
  * @package        Joomla.Build
  *

      ----------- end diff -----------

Found 1 of 2548 files that can be fixed in 57.970 seconds, 42.00 MB memory used
e641a77 3 Feb 2025 avatar laoneo cs
avatar laoneo laoneo - change - 3 Feb 2025
Labels Added: Composer Dependency Changed NPM Resource Changed PR-5.3-dev
avatar joomla-cms-bot joomla-cms-bot - change - 3 Feb 2025
Category External Library Composer Change NPM Change Front End Plugins Repository External Library Composer Change NPM Change Front End Plugins
avatar richard67
richard67 - comment - 3 Feb 2025

The composer update of "joomla/database" brings a new feature (implemented by me in the framework) that you can use custom port numbers in the database host name with PostgreSQL as you already can do with MySQL or MariaDB. Maybe that should be noted for the release announcement?

avatar richard67
richard67 - comment - 3 Feb 2025

Now Drone fails with the "prepare_system_tests" step https://ci.joomla.org/joomla/joomla-cms/81820/1/21 :

+ mv cypress.config.dist.mjs cypress.config.mjs
+ npx cypress install
Unexpected token '.'

Possibly related to the cypress update?

avatar brianteeman
brianteeman - comment - 3 Feb 2025

This is only updating the dependencies withing the restrictions set in the json which may not be sufficient

avatar heelc29
heelc29 - comment - 3 Feb 2025

Possibly related to the cypress update?

Yes, same like here: #44675 (comment)

We could try to use joomlaprojects/docker-images:cypress8.1 also for prepare_system_tests step

Add a Comment

Login with GitHub to post a comment