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

Codecept upgrade #3

Closed
wants to merge 2 commits into from
Closed

Conversation

neclimdul
Copy link

Update codeception to latest version.

@neclimdul
Copy link
Author

Turns out I was having trouble getting codeception to work because I was trying to run 2.0 with php 8 and it was causing weird things. Upgrading everything seems to have fixed it.

Side note, travis' php versions or terribly out of date. Happy to make a pull request for that as well or include it in this one. Just let me know if I should remove the super old versions entirely and replace it with >7.4 or something else.

@thomasnordahl-dk
Copy link
Member

@neclimdul Thank you for your Pull Requests. We are in the process of making sure our packages are PHP 8.1 compatible and will look at your issues regarding codeception and simple-cache ver 2 & 3 at the same time.

We in the Kodus team that will be focusing on this over the upcoming days, so I'm closing these PRs in favor of some PRs from our own team.

Thank your for your inputs, and I apologize for not being able to address these issues earlier :-)

@neclimdul
Copy link
Author

Is there a problem or concern with my code? I tried to keep it to the bare minimum to make the different pieces work so feel to use it as a starting point for your work. Its free under your MIT license.

@thomasnordahl-dk
Copy link
Member

Is there a problem or concern with my code?

No problem with your code at all! :-)

We (the Kodus team) are currently going through all of our repositories and making sure they run on PHP 8.1, and made a quick decision to work on our own branches purely for practical reasons.

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.

None yet

2 participants