Code review comment for ppa-dev-tools:use-subparsers-for-args

Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

Generally ok with this split, with the low amount of suboptions it looks odd, but makes sense IMHO.
One request on better wording or fixing bugs or both (Sorry, didn't track their status if they are already fixed).

Plus one potentially wrong nit pick (you are the native speaker after all).

Either way, none of this is strong enough to stop this +1 overall.

review: Approve

« Back to merge proposal