Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork2k
Allow PHPUnit 10, 11 and 12#3411
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
akrabat commentedSep 20, 2025
Changes look good. Need to sort out the config though. |
williamdes commentedSep 20, 2025
Thanks, it's a bit unexpected |
3f47cb9 tof190e60Comparewilliamdes commentedSep 20, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
@akrabat I improved the GitHub actions workflow, all does pass on my fork That said there is something wrong in the code somewhere. Edit: For example on the merge:https://github.com/slimphp/Slim/actions/runs/17945163370/job/51029735524 PS: bonus, I fixed gitattributes for new added dev stuff |
coveralls commentedSep 23, 2025
18d0fe0 intoslimphp:4.xUh oh!
There was an error while loading.Please reload this page.
akrabat commentedSep 23, 2025
That's really weird as the composer step shows that the Nyholm PSR7 implementation had been installed. |
williamdes commentedSep 23, 2025
Yes and locally when you re run phpunit it may go green |
No backwards incompatible changes