search

Duke-GCB / bespin-cli

Command line client for Bespin
MIT License
0 stars 0 forks source link

simplify command line + workflow/etc creation #25

Closed johnbradley closed 5 years ago

johnbradley commented 6 years ago

Fixes #24 Reworked command line options to be more consistent. Command line organization is based on https://github.com/Duke-GCB/bespin-cli/issues/24#issuecomment-443001611

Some new commands (creating workflows, workflow versions, and workflow configuration) require admin access in bespin-api right now, but this should change when https://github.com/Duke-GCB/bespin-api/issues/172 is finished.

Requires https://github.com/Duke-GCB/bespin-api/pull/178

dleehr commented 5 years ago

In the usage documentation, it's a odd that all of the shortcut aliases are included alongside the full versions, makes it look a lot more complicated:

usage: bespin [-h]
              {workflow,wf,workflow-version,wfv,workflow-config,wfc,share-group,sg,vm-config,vmc,job-template,jt,job,j}
              ...
johnbradley commented 5 years ago

In the usage documentation, it's a odd that all of the shortcut aliases are included alongside the full versions, makes it look a lot more complicated: ...

This is how argparse command aliases look. See aliases=[command.alias], below: https://github.com/Duke-GCB/bespin-cli/blob/e278fe18acc46930cda4e56031690ec5849cbe29/bespin/argparser.py#L31-L37

johnbradley commented 5 years ago

I will look and see if there is a way to adjust the help text.

johnbradley commented 5 years ago

I will look and see if there is a way to adjust the help text.

It doesn't look like there is any easy way to do this. Spoke with @dleehr and will remove the aliases for now.