Skip to content

fix install on php8.1.1 #59

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

Merged
merged 2 commits into from
Dec 30, 2021
Merged

fix install on php8.1.1 #59

merged 2 commits into from
Dec 30, 2021

Conversation

Krakozaber
Copy link
Contributor

@Krakozaber Krakozaber commented Dec 29, 2021

fix codeception/module-yii2 1.1.4 requires php >=5.6.0 <=8.1 -> your php version (8.1.1) does not satisfy that requirement

fix codeception/module-yii2[1.1.2, ..., 1.1.3] require php >=5.6.0 <8.1 -> your php version (8.1.1) does not satisfy that requirement.
@SamMousa
Copy link
Collaborator

Hmm, PHP 8.1.1 should be added to the CI as well then.
Unless GitHub Action uses 8.1 to mean ~8.1 as well.

@Krakozaber
Copy link
Contributor Author

matrix has no separation by minor versions

@Krakozaber
Copy link
Contributor Author

Krakozaber commented Dec 30, 2021

fix composer.json

@Krakozaber Krakozaber mentioned this pull request Dec 30, 2021
Copy link
Collaborator

@SamMousa SamMousa left a comment

Choose a reason for hiding this comment

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

LGTM. Can't see action results from mobile so won't merge it.

@samdark samdark merged commit 14269d0 into Codeception:master Dec 30, 2021
@samdark
Copy link
Member

samdark commented Dec 30, 2021

👍

@dantart
Copy link

dantart commented Jan 9, 2022

Hi! How can I use the latest version with the 8.1.1 change commited ?
I mean, I see the patch presented here, but I can not get it work, since the "source" has not this change officially :-(

@Krakozaber Krakozaber deleted the patch-1 branch January 9, 2022 19:14
@samdark
Copy link
Member

samdark commented Jan 10, 2022

@dantart just tagged release.

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.

4 participants