Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[11.x] PHP 8.4 build #52633

Closed
wants to merge 51 commits into from
Closed

[11.x] PHP 8.4 build #52633

wants to merge 51 commits into from

Conversation

driesvints
Copy link
Member

@driesvints driesvints commented Sep 3, 2024

Blocked by:

@driesvints
Copy link
Member Author

Builds now fail due to a Redis issue:

php: symbol lookup error: /usr/lib/php/20230901/redis.so: undefined symbol: php_rand

@driesvints
Copy link
Member Author

725 deprecations, 10 errors and 8 failures on the PHPUnit v11 build 👀

* Fix implicit null types

* Update some lower requirements

* Fix typo

* More lower version constraints

* More lower version constraints

* For now only run lowest strategy

* Adjust PHPunit version

* Adjust carbon requirement

* Add phpunit 10 again

* Fix phpunit annotations

* Add prefer-stable again, disable fail-fast

* Allow cli-parser 2.0.1 for phpunit 10

* Escape pipe
* Update sebastian/exporter

* Update dependencies for PHP 8.4

* Update dependencies for PHP 8.4

* Update dependencies for PHP 8.4

* run both phpunits version on lowest and stable again

* Disable fail-fast

* Update required predis/predis

* Bump vlucas/phpdotenv

* bump guzzlehttp/promises

* Bump league/flysystem

* wip

* wip

* wip

---------

Co-authored-by: Dries Vints <[email protected]>
@driesvints driesvints changed the title PHP 8.4 build [11.x] PHP 8.4 build Sep 5, 2024
crynobone and others added 19 commits September 20, 2024 08:35
Signed-off-by: Mior Muhammad Zaki <[email protected]>
Signed-off-by: Mior Muhammad Zaki <[email protected]>
Signed-off-by: Mior Muhammad Zaki <[email protected]>
Signed-off-by: Mior Muhammad Zaki <[email protected]>
Signed-off-by: Mior Muhammad Zaki <[email protected]>
Signed-off-by: Mior Muhammad Zaki <[email protected]>
Signed-off-by: Mior Muhammad Zaki <[email protected]>
Signed-off-by: Mior Muhammad Zaki <[email protected]>
Signed-off-by: Mior Muhammad Zaki <[email protected]>
Signed-off-by: Mior Muhammad Zaki <[email protected]>
Signed-off-by: Mior Muhammad Zaki <[email protected]>
Signed-off-by: Mior Muhammad Zaki <[email protected]>
Signed-off-by: Mior Muhammad Zaki <[email protected]>
Signed-off-by: Mior Muhammad Zaki <[email protected]>
Signed-off-by: Mior Muhammad Zaki <[email protected]>
Signed-off-by: Mior Muhammad Zaki <[email protected]>
Signed-off-by: Mior Muhammad Zaki <[email protected]>
Signed-off-by: Mior Muhammad Zaki <[email protected]>
Signed-off-by: Mior Muhammad Zaki <[email protected]>
This reverts commit 4192402.
Signed-off-by: Mior Muhammad Zaki <[email protected]>
Signed-off-by: Mior Muhammad Zaki <[email protected]>
Signed-off-by: Mior Muhammad Zaki <[email protected]>
Signed-off-by: Mior Muhammad Zaki <[email protected]>
Signed-off-by: Mior Muhammad Zaki <[email protected]>
Signed-off-by: Mior Muhammad Zaki <[email protected]>
Signed-off-by: Mior Muhammad Zaki <[email protected]>
.github/workflows/tests.yml Outdated Show resolved Hide resolved
Signed-off-by: Mior Muhammad Zaki <[email protected]>
@pimjansen
Copy link

@driesvints i just bumped fakerphp/faker to 1.24 which includes PHP 8.4 support

@crynobone crynobone marked this pull request as ready for review November 11, 2024 05:08
@crynobone crynobone marked this pull request as draft November 11, 2024 05:08
matrix:
php: [8.2, 8.3]
phpunit: ['10.5', '11.0.1']
php: [8.4]
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there any specific reason why would only support PHP 8.4? Instead just appending to the versioning matrix to include [8.2, 8.3, 8.4].

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is only a temporary testing solution. This is done, so that only the PHP 8.4 tests are running. These PHP versions will be added again after PHP 8.4 tests are passing.

@crynobone
Copy link
Member

crynobone commented Nov 19, 2024

Replaced with #53468

@crynobone crynobone closed this Nov 19, 2024
@crynobone crynobone deleted the php84 branch November 19, 2024 02:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants