-
Notifications
You must be signed in to change notification settings - Fork 233
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 #573
base: 7.x
Are you sure you want to change the base?
Conversation
|
ℹ️ Shift detected you do not have a CI workflow using GitHub Actions. If you would like, Shift can help you get started with the CI Generator. Although the Shift is paid, GitHub Actions is free and having a CI workflow will improve the stability and review process for your package. |
⚗️ 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/sentinel.git"
}
]
} Then update your dependency constraint to reference this branch: {
"require": {
"cartalyst/sentinel": "dev-l11-compatibility",
}
} Finally, run: |
Hi, What are you waiting for? Do you have a plane for approve this pull request for supporting laravel 11 or not? |
Bump Hooman's comment, would be nice to see this go in |
@hooman-mirghasemi @NByam13 - I have just forked this package with support for Laravel 11, that you can install with composer require joshcadman01/cartalyst-sentinel. |
@joshcadman01 much appreciated, I also forked it myself a bit ago and did the upgrade, but thank you for the ping, you're a good dude <3 |
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.