Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Namespace Models #6

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Namespace Models #6

wants to merge 3 commits into from

Conversation

Selembung
Copy link
Owner

This pull request includes changes for namespacing models under App\Models. Feel free to commit any additional changes to the shift-40915 branch.

Before merging, you need to:

  • Checkout the shift-40915 branch
  • Review all pull request comments for additional changes
  • Thoroughly test your application

@laravel-shift
Adopt Laravel coding style
8341c3d 
Shift automatically applies the Laravel coding style - which uses the PSR-2 coding style as a base with some minor additions.

You may customize the adopted coding style by adding your own [PHP CS Fixer][1] `.php_cs` config file to your project root. Feel free to use [Shift's Laravel ruleset][2] to help you get started.

[1]: https://github.com/FriendsOfPHP/PHP-CS-Fixer
[2]: https://gist.github.com/laravel-shift/cab527923ed2a109dda047b97d53c200
@laravel-shift
Shift bindings
78341ad 
PHP 5.5.9+ adds the new static `class` property which provides the fully qualified class name. This is preferred over using class name strings as these references are checked by the parser.
@laravel-shift
Namespace models under App\Models
a1ff645
@Selembung
Copy link
Owner Author

⚠️ While Shift updated model references within your application code, you may reference models outside the code. Be sure to review your environment configuration, database, and queues for additional references.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@Selembung @laravel-shift