RFR: git-pr: use multi-command argument parser
rwestberg at openjdk.java.net
Thu Feb 20 08:35:04 UTC 2020
On Thu, 20 Feb 2020 08:07:38 GMT, Erik Helin <ehelin at openjdk.org> wrote:
> Hi all,
> please review this refactoring of `git pr` to start to use the
> `MultiCommandParser`. This refactoring is needed for several reasons:
> - not all subcommands support all options
> - better help messages
> - better reuse of utility functions
> - the code was essentially one gigantic `main` function
> I plan to split this file into multiple files, one for each command, but that is
> for a later refactoring. I have tried to preserve the semantics of all commands
> and have tested them locally.
Looks good to me!
Marked as reviewed by rwestberg (Reviewer).
More information about the skara-dev