Skip to content

Conversation

@akimd
Copy link
Contributor

@akimd akimd commented Jan 16, 2018

- What I did
I added a \n.

- How I did it
By s/f/ln/

- How to verify it
By docker rmi -f alksdjklasjd, unless you do have an image called this way.

- A picture of a cute animal (not mandatory but encouraged)
porg

Signed-off-by: Akim Demaille <akim.demaille@docker.com>
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 🌮

@codecov-io
Copy link

codecov-io commented Jan 16, 2018

Codecov Report

Merging #810 into master will decrease coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #810      +/-   ##
==========================================
- Coverage   50.92%   50.91%   -0.01%     
==========================================
  Files         237      237              
  Lines       15332    15332              
==========================================
- Hits         7808     7807       -1     
- Misses       7024     7025       +1     
  Partials      500      500

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

@thaJeztah thaJeztah merged commit 3a69e5d into docker:master Jan 16, 2018
@GordonTheTurtle GordonTheTurtle added this to the 18.02.0 milestone Jan 16, 2018
nobiit pushed a commit to nobidev/docker-cli that referenced this pull request Nov 19, 2025
Remove: add missing eol when --force is passed
Upstream-commit: 3a69e5d
Component: cli
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.

5 participants