alces-software / adminware

A sandbox CLI for running commands remotely across nodes
1 stars 0 forks source link

Refactor the `cli` generation and syntax #139

Closed WilliamMcCumstie closed 5 years ago

WilliamMcCumstie commented 5 years ago

This PR finishes off the CLI syntax and generation changes from #133, #136, #137.

It updates the Config commands to use a hash in the command generation process. This makes setting the different elements (e.g. arguments and help text) easier.

It also removes the view tools and view families commands as recommended in #138. This PR removes click_tools and explore_tools as they where no longer being used. This resolves some of the clicky issues and fixes #126.

Removing click_tools and view tools breaks #129, which need to be re-implemented

Caching has also been added to the Config, fixes #114

DavidMarchant commented 5 years ago

Sure thing, LGTM