Update and streamline package for common use case

mpociot / laravel-test-factory-helper

Generate Laravel test factories from your existing models

935 stars 87 forks source link

Update and streamline package for common use case #28

Closed jasonmccreary closed 5 years ago

jasonmccreary commented 5 years ago

Several changes in this, hope you don't mind. Mainly, set the minimum requirement of Laravel 5.5. This allowed removal of PHP < 7 syntax, old references and helpers, and deprecated Symfony methods.

Also did a general scouting of the code which lead to removal of not-so-often used options to help focus the commands purpose.

Finally a rename to provide symmetry with generate commands offered in other packages.

jasonmccreary commented 5 years ago

Know there is a lot here. If you're not interested in maintaining this package anymore I can just use my fork or glad to take over as I have some additional ideas for changes.

mpociot commented 5 years ago

Oh wow...awesome, thanks for looking into the package in the first place. This is one of the very first packages I have ever written for Laravel (you can tell by it not having any tests).

I'm glad that you gave it some love :)