Skip to content
This repository has been archived by the owner on Aug 31, 2023. It is now read-only.

CLI improvements #165

Merged
merged 10 commits into from
Mar 11, 2020
Merged

CLI improvements #165

merged 10 commits into from
Mar 11, 2020

Conversation

sebmck
Copy link
Contributor

@sebmck sebmck commented Mar 11, 2020

This PR improves the CLI help screens. The default rome --help no longer shows command flags, drastically reducing the noise. It also adds command specific --help. Also now when producing diagnostics for flags inside of commands, there's a new INVALID_REQUEST response that's emitted that causes the CLI to dump command usage after the diagnostics are output.

rome --help

Screenshot from 2020-03-10 19-16-45

rome config --help

Screenshot from 2020-03-10 19-16-10

rome config

Screenshot from 2020-03-10 19-15-40

@zinyando
Copy link
Contributor

A few commands are missing a description, we can either have them marked TODO or put their correct description so as to avoid confusion and ambiguity

Screenshot 2020-03-11 at 16 14 38

@sebmck
Copy link
Contributor Author

sebmck commented Mar 11, 2020

Thanks for the suggestion! Just updated the PR.

@sebmck sebmck merged commit 20fd77a into master Mar 11, 2020
@sebmck sebmck deleted the cli-improvements branch March 11, 2020 23:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants