-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
For what version of Laravel this best practices relevant? #82
Comments
in my opinion, if we look at the pattern, it seems to apply to the newest. Like most best practices which are always updated in general. But I'm also curious |
Laravel5 onwards should support all of the practices listed. |
I've created the repository for Laravel 5, but the practices are current. I guess I'll add more tips when I have time. |
Laravel is on a highway in terms of development. I've only started to use L6 and it's L8 already. Maybe it would be best to add version numbers relevant to each practice. BTW it would be great to have an extension for VSCode (and possibly other editors) to lint code based on best practices for used version apart from php common standards linted by phpcs or intelephense. Is there any of the kind being developed? |
People seeing rapid pace of Laravel often get freaked out (especially when they stumble into Jetstream and Tailwind and feel the world has been turned upside down), but the reality is that all of these best practices are equally valid for Laravel 5 through to 8. I've just made the leap from 5 to 8... and after a few months I can confirm that (thankfully) things are NOT all that different. Everything here is still valid. |
Maybe besides adding the "Interface" suffix which probably should be avoided if you take into account Laravel's codebase (vide contracts) or best practices collected here https://xqsit.github.io/laravel-coding-guidelines/docs/naming-conventions/ or https://www.mindtwo.de/guidelines/coding/laravel. There's already plenty of articles about this one: |
For what version of Laravel this best practices?
The text was updated successfully, but these errors were encountered: