Skip to content

Conversation

@tiborvass
Copy link
Collaborator

Signed-off-by: Tibor Vass tibor@docker.com

Signed-off-by: Tibor Vass <tibor@docker.com>
@codecov-io
Copy link

Codecov Report

Merging #1451 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #1451   +/-   ##
======================================
  Coverage    54.2%   54.2%           
======================================
  Files         289     289           
  Lines       19378   19378           
======================================
  Hits        10503   10503           
  Misses       8199    8199           
  Partials      676     676

Copy link
Contributor

@silvin-lubecki silvin-lubecki left a comment

Choose a reason for hiding this comment

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

LGTM, by tuhe way did you check if there is any documentation to update too?

@thaJeztah
Copy link
Member

Looks like there's no documentation yet; https://github.com/docker/cli/tree/master/docs/reference/commandline, so probably we need something as well (can be a follow up)

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 2e38501 into docker:master Oct 17, 2018
@GordonTheTurtle GordonTheTurtle added this to the 19.03.0 milestone Oct 17, 2018
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