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

PHP 8 #58

Closed
wants to merge 1 commit into from
Closed

PHP 8 #58

wants to merge 1 commit into from

Conversation

insekticid
Copy link
Contributor

Your requirements could not be resolved to an installable set of packages.
php_1 |
php_1 | Problem 1
php_1 | - jolicode/elastically[v1.1.0, ..., v1.1.1] require php ^7.2 -> your php version (8.0.2) does not satisfy that requirement.
php_1 | - Root composer.json requires jolicode/elastically ^1.1 -> satisfiable by jolicode/elastically[v1.1.0, v1.1.1].

 Your requirements could not be resolved to an installable set of packages.
php_1             |
php_1             |   Problem 1
php_1             |     - jolicode/elastically[v1.1.0, ..., v1.1.1] require php ^7.2 -> your php version (8.0.2) does not satisfy that requirement.
php_1             |     - Root composer.json requires jolicode/elastically ^1.1 -> satisfiable by jolicode/elastically[v1.1.0, v1.1.1].
@Korbeil
Copy link
Member

Korbeil commented Feb 13, 2021

This won't be enought, you also need to update ruflin/elastica library since current version does not support PHP 8. But there is no version with PHP 8 support at the moment, we have to wait ruflin/Elastica#1871 first.

And please add PHP 8 CI run in order to validate that everything works as expected 😉

@jmsche
Copy link
Contributor

jmsche commented Feb 15, 2021

FYI elastica 7.1.0 was released this morning with PHP 8.0 support.

@damienalexandre
Copy link
Member

Thanks a lot @insekticid - I just merged #59 which resolves the same issue but in a more complete way (with the tests also running on PHP 8).

@insekticid insekticid deleted the patch-1 branch February 17, 2021 12:40
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.

5 participants