Skip to content

Bump moby to f4d4f5863156b82ef146b6ff1e845f8dcf019f12 - #679

Merged
cpuguy83 merged 1 commit into
docker:masterfrom
thaJeztah:revendor-moby
Nov 8, 2017
Merged

Bump moby to f4d4f5863156b82ef146b6ff1e845f8dcf019f12#679
cpuguy83 merged 1 commit into
docker:masterfrom
thaJeztah:revendor-moby

Conversation

@thaJeztah

Copy link
Copy Markdown
MemberAuthor

oh! need to make some changes;

# github.com/docker/cli/cli/trust
cli/trust/trust.go:130: undefined: "github.com/docker/cli/vendor/github.com/docker/docker/registry".DockerHeaders

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@codecov-io

Copy link
Copy Markdown

Codecov Report

Merging #679 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@ Coverage Diff @@## master #679 +/- ##
==========================================
- Coverage 50.49% 50.49% -0.01% 
==========================================
Files 216 216 Lines 17723 17723 ==========================================
- Hits 8950 8949 -1 - Misses 8324 8325 +1 
Partials 449 449

@dnephindnephin left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@cpuguy83cpuguy83 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@cpuguy83
cpuguy83 merged commit 458ce53 into docker:masterNov 8, 2017
@GordonTheTurtleGordonTheTurtle added this to the 17.12.0 milestone Nov 8, 2017
@thaJeztah
thaJeztah deleted the revendor-moby branch November 8, 2017 22:38
Sign up for freeto 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

@thaJeztah@codecov-io@dnephin@cpuguy83@GordonTheTurtle