Skip to content

Conversation

@justynpride
Copy link
Owner

This pull request includes the changes for consolidating custom namespaces. Commit any additional changes to the shift-65698 branch.

Before merging, you need to:

  • Checkout the shift-65698 branch
  • Run composer dump-autoload
  • Review all pull request comments for additional changes
  • Thoroughly test your application

If there were changes you feel could have been automated, please send your feedback to [email protected] or via Twitter.

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 code style applied by adding a [PHP CS Fixer][1] or [PHP CodeSniffer][2] ruleset to your project root. Feel free to use [Shift's Laravel ruleset][3] to help you get started.

For more information on customizing the code style applied by Shift, [watch this short video][4].

[1]: https://github.com/FriendsOfPHP/PHP-CS-Fixer
[2]: https://github.com/squizlabs/PHP_CodeSniffer
[3]: https://gist.github.com/laravel-shift/cab527923ed2a109dda047b97d53c200
[4]: https://laravelshift.com/videos/shift-code-style
@justynpride
Copy link
Owner Author

ℹ️ The following namespaces were not consolidated since their paths were outside of the Laravel app/ folder. If you would like to consolidate any of these namespaces, you may move them within app/ and request a rerun of this Shift.

  • Database\Factories
  • Database\Seeders

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants