Conversation
Codecov Report
@@ Coverage Diff @@
## master #622 +/- ##
=======================================
Coverage 71.83% 71.83%
=======================================
Files 51 51
Lines 2624 2624
=======================================
Hits 1885 1885
Misses 500 500
Partials 239 239 Continue to review full report at Codecov.
|
Signed-off-by: Nicolas De Loof <nicolas.deloof@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@@ -362,7 +362,7 @@ Commands: | |||
push Push an application package to a registry | |||
render Render the Compose file for an Application Package | |||
status Get the installation status of an application | |||
uninstall Uninstall an application | |||
rm Remove an application |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: shouldn't the commands be printed in alphabetical order in the output?
@@ -20,9 +20,10 @@ func uninstallCmd(dockerCli command.Cli) *cobra.Command { | |||
var opts uninstallOptions |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: rename to removeCmd
- What I did
Renamed
uninstall
intorm
(aliasremove
)- How I did it
Search and replace
- How to verify it
docker app --help
docker app rm foo
- Description for the changelog
"uninstall" command renamed to "rm" (alias remove)
- A picture of a cute animal (not mandatory but encouraged)