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

Evaluate move from argparse to click library #44

Open
tzarski0 opened this issue Nov 5, 2024 · 1 comment
Open

Evaluate move from argparse to click library #44

tzarski0 opened this issue Nov 5, 2024 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@tzarski0
Copy link
Contributor

tzarski0 commented Nov 5, 2024

Problem Statement

There are certain limitations with the current argparse cli interface that makes it hard to track dependencies between variables. For example in deploy task if manager_ip = then manager_gateway and manager_mask is required, but when manager_ip = "pat:" then manager_gateway and manager_mask shouldn't be required, but tool still asks for it.

Proposed Solution

Evaluate move to click library to simplify the cli interface. For basic user, basic parameters would be given in cli interface. For power user, all parameters would be specified in the input yaml file (this will be developed later, but we need to make sure new cli will support this in the future).

@tzarski0
Copy link
Contributor Author

tzarski0 commented Nov 5, 2024

Is this something we can solve with this move too? #42

@tzarski0 tzarski0 added the enhancement New feature or request label Nov 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants