search

krakenjs / generator-kraken

Yeoman generator for kraken.js apps
Other
110 stars 59 forks source link

Add config ability to override default partial include #92

Open mikesparr opened 10 years ago

mikesparr commented 10 years ago

When I run the yo kraken:controller <somename> the generator creates the necessary files but I notice it includes /layout/master.dust in templates. If it's not possible (I couldn't find it documented or example), it would be ideal that you could override this default in the config of kraken and generator would consider that for include. Ideally you could then also declare it as command line option to override that default.

The thinking is you may generate an app, build custom layouts, and later as your app expands want to use the generator to add more routes, views, etc. We still have to edit the templates so this is just convenience, but would be nice to have that option as engineers will commonly forget to change that which they didn't code.

jeffharrell commented 10 years ago

I'm not opposed to this. How about something like yo kraken:controller myController --layout=myLayout?

mikesparr commented 10 years ago

Yes, just not sure if the args should be after the name or before. See you have #88 also about adding CLI args so might be related. I'd check other yeoman modules and see how they do it for consistency. It's just convenience but makes sense and custom layouts are very likely. Thanks for considering.