-
Notifications
You must be signed in to change notification settings - Fork 159
Conversation
this.appname = path.basename(process.cwd()); | ||
} | ||
this.appname = this._.slugify(this._.humanize(this.appname)); | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this looks like something that should be in generator core. // @SBoudrias
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, we could automatically check for app name when Generator is instantiated.
What would have priority, bower.json or package.json?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bower.json
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sounds good, is there anything I can pick up with this or are you guys onto it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Feel free to send a PR on the generator system.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, will do...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @p-m-p!
As we've asked for this to go into the core generator system, can this PR be closed now? |
Sure, I'll clean it up and reopen there. |
I commented on an old closed issue but have had to address this myself for a project I am working on and wanted to raise it again here.
I need to be able to set the main application namespace to that of something other than the root directory. The usage section of the Readme implies that this is already supported by supplying an optional name when generating the app but this is ignored.
I've followed the angular generator which uses the app name argument when generating the app and then pulls the name from the bower.json for sub-generator actions. I can add tests if you are willing to pull this in as a feature, if not I suggest changing the docs to prevent other's frustration.