Skip to content
This repository was archived by the owner on Apr 4, 2026. It is now read-only.

Laravel Consolidate Namespaces Shift - #26

Merged
kingRayhan merged 1 commit into
productionfrom
shift-87885
Apr 9, 2023
Merged

kingRayhan merged 1 commit into
productionfrom
shift-87885

Conversation

@kingRayhan

Copy link
Copy Markdown
Member

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

Before merging, you need to:

  • Checkout the shift-87885 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 shift@laravelshift.com or via Twitter.

Shift automatically applies the Laravel coding style - which uses the PSR-12 coding style as a base with some minor additions.

You may customize the code style applied by configuring [Pint](https://laravel.com/docs/pint), [PHP CS Fixer](https://github.com/FriendsOfPHP/PHP-CS-Fixer), or [PHP CodeSniffer](https://github.com/squizlabs/PHP_CodeSniffer) for your project root.

For more information on customizing the code style applied by Shift, [watch this short video](https://laravelshift.com/videos/shift-code-style).
@kingRayhan

Copy link
Copy Markdown
Member 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

@kingRayhan
kingRayhan merged commit 7a97bdb into production Apr 9, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants