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

[RFE] Better CLI argument management - some combinations should not be allowed #556

Open
kambiz-aghaiepour opened this issue Dec 6, 2024 · 0 comments

Comments

@kambiz-aghaiepour
Copy link
Contributor

  • QUADS latest

Describe the bug
Some combinations of CLI arguments should be disallowed, and others are meaningless without supporting arguments.

To Reproduce
Examples of argument combinations that should not be allowe:

    quads --ls-cloud --ls-notification

as well as:

    quads --ls-notification --ls-cloud

In the above two examples, the last one wins. Perhaps there should be better checking for exclusivity of various argument combinations.

Additionally, the following arguments should force requiring using --mod-notification but no such requirement is enforced, so they can be passed (and are ignored if the --mod-notification flag is not passed):

             [--fail {True,False}] [--success {True,False}] [--initial {True,False}]
             [--pre-initial {True,False}] [--pre {True,False}] [--one-day {True,False}] [--three-days {True,False}]
             [--five-days {True,False}] [--seven-days {True,False}]
@sadsfae sadsfae added this to the 2.x Series - Bowie milestone Dec 6, 2024
@sadsfae sadsfae changed the title [BUG] Better CLI argument management - some combinations should not be allowed [RFE] Better CLI argument management - some combinations should not be allowed Dec 14, 2024
@sadsfae sadsfae added enhancement and removed bug labels Dec 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: To do
Development

No branches or pull requests

2 participants