Update dependencies#169
Conversation
There was a problem hiding this comment.
Pull request overview
Updates PHP/Joomla-related dependencies and devcontainer defaults to newer patch/minor versions, aligning the development/tooling stack with current upstream releases.
Changes:
- Bumped Joomla CMS from 6.0.2 to 6.0.3 across Composer metadata and devcontainer Docker build args.
- Updated dev tooling dependencies in
composer.lock(notablyfriendsofphp/php-cs-fixer, PHPStan, Symfony components). - Updated the Joomla 5 devcontainer override to use Joomla 5.4.3.
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| composer.lock | Refreshes locked versions for dev tooling (php-cs-fixer, PHPStan, Symfony, etc.) and Joomla CMS package entry. |
| composer.json | Updates declared dev dependency constraints and Joomla CMS package repository metadata to 6.0.3. |
| .devcontainer/joomla/Dockerfile | Updates the default Joomla 6 image version used by the devcontainer build. |
| .devcontainer/compose.joomla5.yml | Updates the Joomla 5 override to pull/build Joomla 5.4.3. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| @@ -44,8 +44,8 @@ | |||
| "php": ">=8.1" | |||
| }, | |||
| "require-dev": { | |||
There was a problem hiding this comment.
composer.json still declares PHP >=8.1, but the updated dev dependency set (via php-cs-fixer 3.94 / sebastian-diff 8 / Symfony 8 pulled in by the lockfile) requires PHP >=8.4. As a result, composer install (with dev deps) will fail on PHP 8.1–8.3 even though the root constraint suggests it should work. Consider either (a) raising the root PHP requirement to match the tooling baseline, or (b) explicitly adding a php: >=8.4 constraint under require-dev and updating contributor docs so the runtime requirement and dev-tooling requirement are clearly separated.
| "require-dev": { | |
| "require-dev": { | |
| "php": ">=8.4", |
Remove resolved Text::sprintf error suppression.
3a3c039 to
c575624
Compare
No description provided.