-
Notifications
You must be signed in to change notification settings - Fork 10
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.
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
[1.2] Refactoring stubs #71
Conversation
Hi, do you have an approximate release date for version 1.2? :) |
Hey Nathan, sorry for the delay and thanks for your patience! If you're not too concerned about potential issues with stubs, you can start using 1.2 like this: composer require lunarstorm/laravel-ddd:dev-next You might have to set composer.json's minimum-stability to "beta" I dropped the v1.2.0 pre-release recently, https://github.com/lunarstorm/laravel-ddd/releases/tag/v1.2.0-beta.1 - what is the composer convention to use pre-releases? I wasn't sure. Either way, targeting |
Changes in #69 require stubs to be refactored before 1.2 can be officially tagged and released.
.stub
extension instead of.php.stub
(following Laravel's convention)ddd:model
extends laravel'smake:model
and therefore inherits the native stubs.base_path('stubs/ddd')
instead ofresource_path('stubs/ddd')
. In other words, the ddd stubs will now be co-located alongside the framework's published stubs, within addd
subfolder.stubs/ddd/*.stub
(allows customizing stub for ddd only)stubs/*.stub
(shared by both make:* and ddd:*)Todo
ddd:publish
to provide a more intuitive way to publish stubs selectively. Currently, to customize addd:model
stub, you would have to publish all of laravel's stubs and customize themodel.stub
file, or make a copy of it into the ddd subfolder. The ideal workflow would be a way to publish ONLY the stub you need, and have it end up instubs/ddd/*.stub
automatically.