-
-
Notifications
You must be signed in to change notification settings - Fork 450
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
[4.x] Laravel 12 support #1321
[4.x] Laravel 12 support #1321
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1321 +/- ##
=========================================
Coverage 84.74% 84.74%
- Complexity 1063 1071 +8
=========================================
Files 177 177
Lines 3100 3101 +1
=========================================
+ Hits 2627 2628 +1
Misses 473 473 ☔ View full report in Codecov by Sentry. |
@stancl Why did you drop support for Laravel 11? |
It is easier to update the average app to Laravel 12 than for us to maintain compatibility with both L11 and L12. |
There are still a lot of other packages that don't support L12, Now I can't update my tenancy projects. |
You can lock the tenancy version to an older commit that supports Laravel 11 and only switch back to dev-master when you can update your entire app. We'll soon be pushing changes that will likely depend on changes made to Laravel 12 — it would make no sense to support Laravel 11. |
When I lock it to an older commit, composer will still look at the latest |
This PR adds Laravel 12 support and drops support for all other Laravel versions.