- Notifications
You must be signed in to change notification settings - Fork13
Add laravel v12 to CI#57
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
0cde5dc to316fce5CompareHi@asanikovich, It’s not enough to just update our continuous integration file to add a new version. There must be a branch in theLaravel 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 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. |
Ops, It seems like I wrote it too quickly!! 🤣🤣 |
000b46b intoCodeception:mainUh oh!
There was an error while loading.Please reload this page.
@asanikovich It seems that the version of https://github.com/Codeception/laravel-module-tests/tree/12.x However, everything is fine here. Thank you! |
@TavoNiievezCodeception/laravel-module-tests#24 I have updated actions/cache |
This MR updates our CI to include testing against Laravel 12. It makes the CI matrix aware of the new framework version.