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
Feature: interactive meteor create command #12803
Feature: interactive meteor create command #12803
Conversation
…ure/interactive-meteor-create-command
…ithub.com/mr-loop-1/meteor into feature/interactive-meteor-create-command
Hi @StorytellerCZ , |
I added testing this PR to my todo's |
tools/cli/commands.js
Outdated
const skeletonExplicitOption = AVAILABLE_SKELETONS.find(skeleton => | ||
!!options[skeleton]); | ||
|
||
if(skeletonExplicitOption) { |
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.
If a name is provided as an arg, the interactivity should be disabled and it use the default skeleton if non was provided. This is needed to avoid breaking scripts that use meteor create
.
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.
@zodern Yes, that would be one of the cases where we wouldn't want to make prompts.
So, in that case, we only bring up the prompts when app name is not provided?
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.
@zodern Alternatively, we could have a simple info line for users when they provide an app name but no template. It could go along the lines of - no skeleton specified, using default React....
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.
So, in that case, we only bring up the prompts when app name is not provided?
Yes.
Showing a message like you suggest when using the default skeleton makes sense.
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.
@zodern made some changes
This comment was marked as off-topic.
This comment was marked as off-topic.
Needs a new dev-bundle. cc @StorytellerCZ |
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.
LGTM
Closes #12791
If either or both of application name and skeleton flag is missed in the create command, there will be prompts for choosing them.