Re: [PATCH] Changed error message when morph deploy is given too few arguments
On Thu, Mar 21, 2013 at 10:37:02AM +0000, Jonathan Maw wrote:
I wonder if it would be better to just tell them to check the
command-line from
`morph --help`, and make that help information more informative.
Currently the
help for deploy just says "Deploy a built system image" without
explaining what
TYPE, SYSTEM and LOCATION are. Either way, LOCATION is currently
under-documented and may require talking to the spirits to get the
correct text.
I think Jonathan is right: the right place for this is in the
subcommand's help. This is easy to do: add to the docstring to the
"deploy" method in DeployPlugin. It currently says just
"Deploy a built system image", and should include the extra text
you added to the error message. Could you prepare a new patch that
does that, please, Ric?
--
http://www.codethink.co.uk/ http://wiki.baserock.org/ http://www.baserock.com/