-
-
Notifications
You must be signed in to change notification settings - Fork 89
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 11.x Compatibility #98
Conversation
|
|
⚗️ Using this package? If you would like to help test these changes or believe them to be compatible, you may update your project to reference this branch. To do so, temporarily add Shift's fork to the {
"repositories": [
{
"type": "vcs",
"url": "https://github.com/laravel-shift/laravel-visits.git"
}
]
} Then update your dependency constraint to reference this branch: {
"require": {
"awssat/laravel-visits": "dev-l11-compatibility",
}
} Finally, run: |
Tests failed need more work |
My guess is these are dependency/CI related. For example, there isn't a |
When this will publish? |
any news? This is my only package thats keeping me from upgrading to 11 right now. |
@Jerimu PRs are always welcome |
This is a PR for L11 compability |
yeah but tests are failing sorry I don't have time to fix this PR. |
Latest release supports Laravel 11 now. |
PR forgot to add Carbon ^3.0 in the depedencies for new created L11 projects - see #99 |
Laravel 11 supports both versions 2 and 3. |
Yes and new projects comes with 3, as I tried to require 6.1 in a fresh L11 project. |
This is an automated pull request from Shift to update your package code and dependencies to be compatible with Laravel 11.x.
Before merging, you need to:
l11-compatibility
branchIf you do find an issue, please report it by commenting on this PR to help improve future automation.