Laravel Consolidate Namespaces Shift #50

Merged
nomadjimbob merged 1 commits from shift-91887 into main 2023-05-24 21:38:32 +00:00
nomadjimbob commented 2023-05-24 21:36:44 +00:00 (Migrated from github.com)

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

Before merging, you need to:

  • Checkout the shift-91887 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.

This pull request includes the changes for consolidating custom namespaces. Commit any additional changes to the `shift-91887` branch. **Before merging**, you need to: - Checkout the `shift-91887` 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](mailto:shift@laravelshift.com) or via [Twitter](https://twitter.com/laravelshift).
nomadjimbob commented 2023-05-24 21:36:45 +00:00 (Migrated from github.com)

ℹ️ 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
  • Faker\Provider
:information_source: 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 - [ ] Faker\Provider
Sign in to join this conversation.