-
Notifications
You must be signed in to change notification settings - Fork 904
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
Improve overall interface #36
Comments
I really like "Yargs" and the way it's set up in Metro CLI. I am thinking this should help us simplify the overall interface even more and get rid of custom "parsing" functions that are causing an unnecessary maintenance. |
Observation as I am fixing ESLint issues: Many commands console.log, others use "log" and others "npmlog". Some return errors, some print errors and return nothing. This should be unified as a first starting point in order to support proper UI. |
This is mostly done now. What we need now is small incremental adjustments to the UI, no need to track it in this ticket. |
The CLI experience is very
raw
. We should analyse what other CLI tools do in that space (Yarn, Jest and more - suggestions are welcome) and try to come up with our plan.I've been working quite heavily on Haul where we did a "Create React App" - like interface that takes up the whole available space and cleans up the terminal when e.g. the build finishes.
I am leave this issue here open for discussion - would love to gather some ideas before we get started.
The text was updated successfully, but these errors were encountered: