Code review comment for lp:~waigani/juju-core/cmd_help_aliases_1299120

Revision history for this message
Jesse Meek (waigani) wrote :

Reviewers: mp+218728_code.launchpad.net,

Message:
Please take a look.

Description:
Fix 1299120 command aliases

Fix bug 1299120 by ensuring that the
preferred command is not displayed as
an alias of another command in juju
help commands.

Change file and function names to
be consistant with the preferred
command.

https://code.launchpad.net/~waigani/juju-core/cmd_help_aliases_1299120/+merge/218728

(do not edit description out of merge proposal)

Please review this at https://codereview.appspot.com/92080046/

Affected files (+149, -140 lines):
   A [revision details]
   M cmd/juju/cmd_test.go
   M cmd/juju/main.go
   M cmd/juju/removemachine.go
   M cmd/juju/removemachine_test.go
   M cmd/juju/removerelation.go
   M cmd/juju/removerelation_test.go
   M cmd/juju/removeservice.go
   M cmd/juju/removeservice_test.go
   M cmd/juju/removeunit.go
   M cmd/juju/removeunit_test.go
   M environs/manual/provisioner.go
   M state/api/client.go
   M state/api/params/params.go
   M state/apiserver/client/client.go
   M state/apiserver/client/client_test.go
   M state/apiserver/client/perm_test.go
   M state/minimumunits_test.go
   M state/relation_test.go
   M state/relationunit_test.go
   M state/unit_test.go

« Back to merge proposal