- Issue created by @kim.pepper
- ๐ฎ๐ณIndia rajeshreeputra Pune
- ๐ซ๐ทFrance andypost
- Merge request !5725Upgrade phpspec/prophecy to v1.18.0 to support PHP 8.3 #3406622 โ (Closed) created by andypost
- Status changed to Needs review
about 1 year ago 4:46pm 7 December 2023 - Status changed to RTBC
about 1 year ago 5:00pm 7 December 2023 - ๐บ๐ธUnited States smustgrave
Thanks @andypost for keeping up with all these 8.3 issues!
- ๐ฌ๐งUnited Kingdom longwave UK
Guessing we did not catch this in GitLab CI because we run
composer install
on the default version of PHP (currently 8.2), and then reuse the same vendor directory artifact on the real PHP version in the tests? Should we change the pipeline to ensure thatcomposer install
is successful on all supported versions? - Merge request !5727Draft: Test #3406622 "Phpspec prophecy 1 18 test" โ (Closed) created by andypost
- ๐ซ๐ทFrance andypost
Opened MR to test but then figured out that composer checking on
phpspec/prophecy-phpunit
which require prophecy itselfVery probably composer does not check platform reqs for indirect dependencies
- ๐ฎ๐นItaly mondrake ๐ฎ๐น
+1, also needed for PHPUnit 10 support. Bumping to critical, we need to have this in 10.2.0 I assume.
- ๐ฌ๐งUnited Kingdom longwave UK
As this is just a lockfile change I think we should roll this into ๐ Update composer dependencies for Drupal 10.2.0 Active where we are going to do a final round of all PHP dependency updates.
- Status changed to Closed: duplicate
about 1 year ago 6:50pm 7 December 2023 - ๐ซ๐ทFrance andypost
Added changes to ๐ Update composer dependencies for Drupal 10.2.0 Active