remove the status command from docker app CLI #647
remove the status command from docker app CLI #647
Conversation
e2e/commands_test.go
Outdated
@@ -316,36 +316,10 @@ func testDockerAppLifecycle(t *testing.T, useBindMount bool) { | |||
|
|||
// Query the application status | |||
cmd.Command = dockerCli.Command("app", "status", appName) |
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.
We can remove this too I think.
It reads weirdly for newcomers, why would we check for a random app command and look that the output is that this command is not defined?
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.
👌
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.
I agree 👍
Codecov Report
@@ Coverage Diff @@
## master #647 +/- ##
=========================================
Coverage ? 71.35%
=========================================
Files ? 48
Lines ? 2451
Branches ? 0
=========================================
Hits ? 1749
Misses ? 480
Partials ? 222
Continue to review full report at Codecov.
|
89717de
to
da6fa10
Compare
There are a bunch of references to the |
Signed-off-by: Guillaume Lours <guillaume.lours@docker.com>
da6fa10
to
5a52b2f
Compare
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
Signed-off-by: Guillaume Lours guillaume.lours@docker.com
- What I did
Remove status command from docker-app CLI
- How I did it
Remove code
- How to verify it
docker app status
, check it displays- Description for the changelog
status
command has been removed from the CLI- A picture of a cute animal (not mandatory but encouraged)