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.

Sign up for GitHub

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

Jump to bottom

PHP 8.1 support / composer #531

Closed
alecpl opened this issue Jul 25, 2021 · 2 comments · Fixed by #533
Closed

PHP 8.1 support / composer #531

alecpl opened this issue Jul 25, 2021 · 2 comments · Fixed by #533

Comments

@alecpl
Copy link

alecpl commented Jul 25, 2021 •
edited
Loading 

- phpunit/phpunit[9.4.1, ..., 9.5.7] require phpspec/prophecy ^1.12.1 -> satisfiable by phpspec/prophecy[1.12.1, 1.12.2, 1.13.0].
...
- phpspec/prophecy[1.12.0, ..., 1.13.0] require php ^7.2 || ~8.0, <8.1 -> your php version (8.1.0-dev) does not satisfy that requirement.

It would be nice if we could test our code before the 8.1.0 is released.
alecpl added a commit to roundcube/roundcubemail that referenced this issue Jul 25, 2021
@alecpl
GA: Remove PHP 8.1 from the matrix
5592f5d 
Some deps aren't compatible yet phpspec/prophecy#531
@dpi dpi mentioned this issue Jul 28, 2021
Merged
dpi added a commit to dpi/dogit that referenced this issue Jul 28, 2021
@dpi
remove php8.1 due to phpspec/prophecy#531
dedbff1
@javer javer mentioned this issue Jul 29, 2021
Merged
@stloyd stloyd mentioned this issue Aug 2, 2021
Merged
@BenMorel
Copy link

BenMorel commented Aug 2, 2021 •
edited
Loading

+1. This library being a dependency of PHPUnit, most open-source libraries can't be tested on PHP 8.1 before Prophecy is declared compatible.

Now that PHP 8.1 has entered beta, I'm eager to test my libraries against it.

Looking forward to #533 being merged & released!

@chris-doehring
Copy link

I suggest to reopen this issue since there is no php 8.1 release, yet. For those who want to know about the current status:

#533 (comment)

I've added some issues for things we'll need to check on PHP 8.1

https://github.com/phpspec/prophecy/issues?q=is%3Aopen+is%3Aissue+label%3APHP8.1

Any extra issues people can think of, please raise them

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add PHP 8.1 support javer/prophecy
3 participants
@alecpl @BenMorel @chris-doehring