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.

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 README / Don't print usage errors when using -h/--help #9

Merged
merged 2 commits into from
Mar 30, 2020

Conversation

nastra
Copy link

@nastra nastra commented Mar 30, 2020

Currently when you do mgmtapi --help you'll also see a usage error, because a required option wasn't provided:

mgmtapi --help

    Usage error: Required option '-S' is missing

    NAME
            cassandra-management-api - REST service for managing an Apache
            Cassandra node

This is unfortunately due to a bug in airline itself (airlift/airline#44). So it's easier to just have a workaround in place when printing the help message.

Eduard Tudenhoefner added 2 commits March 30, 2020 17:57
This is unfortunately due to a bug in airline itself
(airlift/airline#44). So it's easier to just
have a workaround in place when printing the help message.
@nastra nastra requested a review from tjake March 30, 2020 16:02
Copy link
Contributor

@tjake tjake left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great, thanks!

@tjake tjake merged commit 9c25b81 into master Mar 30, 2020
@tjake tjake deleted the ET-impr branch March 30, 2020 17:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants