Skip to content

Conversation

asanikovich
Copy link
Contributor

This MR updates our CI to include testing against Laravel 12. It makes the CI matrix aware of the new framework version.

@TavoNiievez
Copy link
Member

Hi @asanikovich,

It’s not enough to just update our continuous integration file to add a new version. There must be a branch in the Laravel Module Test project with this version installed, and we need to have verified that the tests with this module pass on that branch.

So you’ve done only half the work here: you still need to open a pull request in the other repository, based on branch 11.x and updating it to version 12. Once you do that, I’ll create the new branch when I merge that pull request.

Only then will the tests for this pull request pass.

Additionally, Laravel 10 is no longer supported, so we should only support two versions in this module —11 and 12.
That’s another change we should make.

@asanikovich
Copy link
Contributor Author

@TavoNiievez
Copy link
Member

Ops, It seems like I wrote it too quickly!! 🤣🤣

@TavoNiievez TavoNiievez merged commit 000b46b into Codeception:main Sep 26, 2025
15 of 18 checks passed
@TavoNiievez
Copy link
Member

@asanikovich It seems that the version of actions/cache needs to be updated:

https://github.com/Codeception/laravel-module-tests/tree/12.x

However, everything is fine here.

Thank you!

@asanikovich
Copy link
Contributor Author

@TavoNiievez Codeception/laravel-module-tests#24 I have updated actions/cache

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.

2 participants