Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Rename flag --migrate to a command (migrate) #89

Closed
okonet opened this issue Mar 18, 2017 · 2 comments
Closed

Rename flag --migrate to a command (migrate) #89

okonet opened this issue Mar 18, 2017 · 2 comments
Labels
AST Transform CLI enhancement

Comments

@okonet
Copy link
Contributor

okonet commented Mar 18, 2017

To make it more consistent with other CLI tools I'd propose renaming webpack --migrate to webpack migrate. Historically, -- are for options but migrate is a command. So, the usage would look like

webpack migrate path/to/webpack.config.js

This would also allow us passing some arbitrary options to this command (like, code style for prettier etc.) in the future.
@evenstensberg
Copy link
Member

I agree, same goes for init

okonet added a commit that referenced this issue May 4, 2017
@okonet
Better CLI
090977d 
- Rename migrate and init options to command. Closes #89
- Use `commands` from yargs
- Add start command and made it default
- Move all options to one place
- Show options only for start command
@evenstensberg
Copy link
Member

Ongoing in #121 , merging to #105 later, closing for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
AST Transform CLI enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@okonet @evenstensberg