Proposal: leveraging DocOpt subcommands about molecule

[retr0h]

Totally agree.

/cc @bborysenko @abrown-sg

[rgreinho]

@retr0h Even with a way better command dispatcher to get rid of the if ... elif ... else that you dislike :)

https://github.com/rgreinho/docopt-subcommands-example#find-the-command-to-execute
https://github.com/rgreinho/docopt-subcommands-example/blob/master/main.py#L37-L43

[retr0h]

@retr0h Even with a way better command dispatcher to get rid of the if ... elif ... else that you dislike :)

[identifystation]

The challenge we're facing when trying to implement this (which may be a DocOpt bug) is that long arguments don't seem to work in both forms. For example:

--platform=ubuntu WORKS
--platform ubuntu FAILS

The expected behavior is that both will work in DocOpt, however we've been unable to find a workaround with Remy's subcommand pattern that support this.

#81 implements as much of this as possible, however, by moving commands into individual classes. This is a win regardless since it's a significant simplification over what we were using previously. It will also be easy to drop in DocOpt subcommand support once the bug is resolved.

[rgreinho]

I filed an issue upstream: docopt/docopt#305.

[rgreinho]

@abrown-sg There was a mistake in my tutorial (that I fixed), and now the parameters for the arguments work both with and without the equal sign.

Basically there need to be an ellipsis ('...') after the args (looks like [<args>...]), so that the program can have multiple values. The issue was that --platform ubuntu was counted as 2 arguments, but as the ellipsis was not specified, only one argument was allowed.

If you can confirm that it works for your branch as well (I don't think you pushed it, so I could not try myself), then I'll close the issue that I filled upstream.