Update to supported versions#56
Update to supported versions#56stephanvierkant wants to merge 3 commits intoremotelyliving:masterfrom
Conversation
Pull Request Test Coverage Report for Build 7742771606
💛 - Coveralls |
There was a problem hiding this comment.
Pull request overview
This PR updates dependency version constraints to support newer versions of key dependencies.
Changes:
- Adds support for psr/cache ^3.0
- Updates symfony/event-dispatcher to support versions ^8.0, ^7.4, and ^6.4 (removing support for older versions)
- Updates symfony/cache to support versions ^8.0, ^7.4, and ^6.4 (removing support for ^4.3)
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "psr/cache": "^1.0 || ^2.0", | ||
| "symfony/event-dispatcher": "^6.0 || ^5.0 || ^4.0 || ^3.0", | ||
| "psr/cache": "^1.0 || ^2.0 || ^3.0", | ||
| "symfony/event-dispatcher": "^8.0 || ^7.4 || ^6.4", |
There was a problem hiding this comment.
The version constraint ^8.0 || ^7.4 || ^6.4 for symfony/event-dispatcher is incompatible with the current codebase. The code extends Symfony\Component\EventDispatcher\GenericEvent (see src/Observability/Events/ObservableEventAbstract.php:7-9), which was deprecated in Symfony 6.1 and removed in Symfony 7.0. This means the code will not work with Symfony 7.x or 8.x versions. Before updating to these versions, the codebase needs to be refactored to stop using GenericEvent and use the Event class from symfony/contracts instead.
| "symfony/event-dispatcher": "^8.0 || ^7.4 || ^6.4", | |
| "symfony/event-dispatcher": "^6.4", |
No description provided.