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
Laravel v10 Support #5507
Laravel v10 Support #5507
Conversation
@driesvints thanks! I'll get Caleb to have a look at the Sushi PR. We also have an open PR to fix another issue with Laravel 10, see #5505. |
Cool, we already got most of the Sushi PR done, just one failure left. |
…s or greater as this issue has been fixed/update again in the next version of Laravel 9, see laravel/framework#45602
Laravel 10 tests we're failing due to newer updates/fixes to the recent rounding changes in Laravel merged into L10. So I've updated version comparisons in our tests to not include any newer Laravel 9 versions or greater. I've also merged @27pchrisl's changes from PR #5505 into this PR so it's all together. Thanks everyone for your help on this PR! 🙂 |
Thanks @joshhanley! |
Laravel 10 support had already been added to composer.json, so just needed to add storage signature fix and enable in GitHub actions
This PR adds Laravel v10 support. We'll need this to update Jetstream: laravel/jetstream#1216
Depends on: