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

Move to parent parser to avoid separated command line arguments #85

Open
tvogels01 opened this issue Jan 5, 2018 · 0 comments
Open

Comments

@tvogels01
Copy link
Contributor

Summary

Current:

arthur.py --submit $CLUSTER_ID extract --prolix

Desired:

arthur.py extract --submit $CLUSTER_ID --prolix

Brownie points when --submit automatically turns on "prolix" mode and (maybe?) sets the "prefix".

Details

This is most apparent when submitting jobs to the cluster where "help" may not do what a user might reasonably expect:

arthur.py --submit $CLUSTER_ID extract --help

Now you have to find the help in the output of the command on the cluster.

This would be clear (and print the help message for extract in terminal):

arthur.py extract --submit $CLUSTER_ID --help

Having the --submit and --config args after the verb would also make command line expansion easier.

It gets trickier when considering "dry-run":

arthur.py  extract --submit $CLUSTER_ID --dry-run

For now, this should indeed submit "extract --dry-run". Consider the workflow:

arthur.py  upgrade --from dw.user --dry-run
arthur.py  upgrade --from --submit $CLUSTER_ID

(Which is almost great .. but might need --submit to imply --prolix and --prefix {whatever prefix is configured})

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant