Skip to content

Conversation

@e11137
Copy link

@e11137 e11137 commented Jun 6, 2017

Always exit with 0 if the -f force option is used

@codecov-io
Copy link

codecov-io commented Jun 8, 2017

Codecov Report

Merging #160 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #160      +/-   ##
==========================================
+ Coverage   45.36%   45.37%   +<.01%     
==========================================
  Files         171      171              
  Lines       11456    11458       +2     
==========================================
+ Hits         5197     5199       +2     
  Misses       5962     5962              
  Partials      297      297

@dgageot
Copy link
Member

dgageot commented Jun 13, 2017

Hi @e11137, you have to sign your commit for Gordon the Turtle to like you!

Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@@ -0,0 +1,2 @@
NotFound:image1
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Was discussing with @vdemeester that the naming of these files is a bit odd; perhaps we should normalise them to be lowercase and have spaces replaced with -. Given that these files are generated, and this is in line with other files in this tests, we can keep it for now 👍

@thaJeztah
Copy link
Member

@dgageot it was already fixed

screen shot 2017-06-13 at 13 43 32

@docker docker deleted a comment from GordonTheTurtle Jun 13, 2017
Copy link
Collaborator

@vdemeester vdemeester left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@vdemeester vdemeester merged commit ecc8e0a into docker:master Jun 13, 2017
@GordonTheTurtle GordonTheTurtle added this to the 17.07.0 milestone Jun 13, 2017
@thaJeztah
Copy link
Member

Thank you @e11137 !

trapier pushed a commit to trapier/cli that referenced this pull request Sep 30, 2019
…-19.03-2f1931f9eb2d6bac2efd48d94739f2e9919d4d7d

[19.03] sync to upstream 19.03 2f1931f
nobiit pushed a commit to nobidev/docker-cli that referenced this pull request Nov 19, 2025
[17.06.1] bump version to 17.06.1-ce-rc3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants