Michael Pasternak has posted comments on this change. Change subject: cli: Change action help text to match syntax ......................................................................
Patch Set 1: > These changes don't break backward compatibility. They > relax the command parser so that the position of the > options isn't relevant. The parser accepts the old syntax > and the new one. after reverting [2], old syntax doesn't work for actions on objects which are not top-level. > This change in particular only changes the documentation > so that it matches what the user actually has to type to > make autocompletion work correctly. user should be able using old syntax that you omitting, you forgetting that command can come not only from autocompletion, but from script or just handcrafted using old syntax. -- To view, visit http://gerrit.ovirt.org/24081 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ib3d82efa4a7fbff4ff04cb9a2c0d7b9fec3670e8 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <[email protected]> Gerrit-Reviewer: Juan Hernandez <[email protected]> Gerrit-Reviewer: Michael Pasternak <[email protected]> Gerrit-Reviewer: Ravi Nori <[email protected]> Gerrit-HasComments: No _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
