-
Notifications
You must be signed in to change notification settings - Fork 159
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
props or just variable #38
Comments
I would vote for all props. |
One question. How do I support variable like |
In what scenario it won't be from |
But sometimes we need other In my opinion, maybe In the template, we don't need But we need to assign the variable from props in the |
In that case the variables are a good solution. What says @sindresorhus @SBoudrias ? |
Total 10 props, I think more than 10 will cause dislike 😢 var prompts = [{
name: 'description',
message: 'Description',
default: 'The best module ever.'
}, {
name: 'homepage',
message: 'Homepage'
}, {
name: 'license',
message: 'License',
default: 'MIT'
}, {
name: 'githubUsername',
message: 'GitHub username'
}, {
name: 'authorName',
message: 'Author\'s Name'
}, {
name: 'authorEmail',
message: 'Author\'s Email'
}, {
name: 'authorUrl',
message: 'Author\'s Homepage'
}, {
name: 'keywords',
message: 'Key your keywords (comma to split)'
}, {
name: 'cli',
message: 'Do you need cli tools?',
default: 'yes'
}, {
name: 'browser',
message: 'Do you need browserify?',
default: 'yes'
}]; |
We could just stick on the github user name or author name and skip email and URL? |
I think |
👍 So be it? Update: @huei90 Lets reduce it to bare minimal. |
Hi guys, I want fix some trailing spaces in package.json. |
@faustman Go ahead, this is yet to be finalized. |
@faustman Go Ahead! |
@huei90 If you notice the latest |
no action from me, |
👍 |
In the template file
for example
In this case we have variable
slugname
props.homepage
Can we select just one type
props
or
variable
(assign all props to a variable, letprops.authorName
=>authorName
)Easier for maintenance.
Thanks
The text was updated successfully, but these errors were encountered: