-
-
Notifications
You must be signed in to change notification settings - Fork 283
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 7 support #368
Comments
@leasophiiie Why you say that there is no support? |
Because of this
|
Yeah, looks like |
On a fresh install of laravel7 (ie
To fixFirst, upgrade phpunit to v9.0:
Then, you can install phpinsights:
For Symfony5 usersUnlike laravel, symfony/skeleton or symfony/website-skeleton came with a minimum-stability to To allow install all dependencies on SF5, you should allow a RC minimum-stability:
Then you can install phpinsights.
|
@Jibbarth auch, that's not the most user friendly way. Maybe we should look into if we can drop composer dependency? |
@olivernybroe I already tried to rewrite the validate part of ComposerMustBeValid but it's a huge feature 😞 Maybe, as it was proposed on your PHPStan PR, we could parse the result of For the ComposerLockMustBeFresh, I think it's easier to recode, because it's based on a hash (don't remember which type) of the But drop composer dependency will not fix the conflict in laravel7 between phpunit:8.5 dependencies and phploc:6.0 dependencies (ie sebastian/version 🙁 ) |
When will this be released -- so that we don't need to use |
I would like to know, what about the support for Laravel 7? @nunomaduro @Jibbarth @olivernybroe
The text was updated successfully, but these errors were encountered: