-
Notifications
You must be signed in to change notification settings - Fork 138
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
Shorter options help #1872
Shorter options help #1872
Conversation
modules/specification-level/src/main/scala/scala/cli/commands/SpecificationLevel.scala
Outdated
Show resolved
Hide resolved
- shorten the help message - tweak which options are included in the short help - misc tweaks
- shorten the help message - tweak which options are included in the short help - misc tweaks
- shorten the help message - tweak which options are included in the short help - misc tweaks
- shorten the help message - tweak which options are included in the short help - misc tweaks
- shorten the help message - tweak which options are included in the short help - misc tweaks
- shorten the help message - tweak which options are included in the short help - misc tweaks
- shorten the help message - tweak which options are included in the short help - misc tweaks
…blish`, `publish local` and `publish setup` sub-commands
…cret create` sub-commands
- temporarily remove console-specific characters from help messages to fix the reference doc website - remove redundant tags for options already tagged as `SpecificationLevel.MUST` - rename `tags.important` to `tags.inShortHelp`
a64298f
to
944acd4
Compare
Btw. as a side effect of these changes, I had to adjust the behaviour of old output:
new output:
This probably should be tweaked further separately at a later date. |
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
010cc83
to
cfc88b1
Compare
modules/cli/src/main/scala/scala/cli/commands/export0/ExportOptions.scala
Show resolved
Hide resolved
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.
Great work! I left a very minor comment for a future refactor
Fixes #1669
scala-cli-signing
andcase-app
tags.important
ortags.must
help options in the short help (sub-command --help
)