close
Skip to content

Bump Go to 1.9.2 - #716

Merged
dnephin merged 1 commit into
docker:masterfrom
vdemeester:bump-go
Nov 29, 2017
Merged

Bump Go to 1.9.2#716
dnephin merged 1 commit into
docker:masterfrom
vdemeester:bump-go

Conversation

@vdemeester

Copy link
Copy Markdown
Collaborator

Now that moby/moby#33892 is merged, we can update docker/cli to go 1.9.2 too 👼

🐯 🐻 🦁

Signed-off-by: Vincent Demeester vincent@sbr.pm

Signed-off-by: Vincent Demeester <vincent@sbr.pm>
@@ -1,3 +1,3 @@
FROM dockercore/golang-cross@sha256:25ff84377e9d7f40639c33cc374166a3b0f1829b8462cf7001d742a846de2687
FROM dockercore/golang-cross@sha256:2e843a0e4d82b6bab34d2cb7abe26d1a6cda23226ecc3869100c8db553603f9b

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

@codecov-io

codecov-io commented Nov 29, 2017

Copy link
Copy Markdown

Codecov Report

Merging #716 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #716      +/-   ##
==========================================
- Coverage   51.65%   51.64%   -0.01%     
==========================================
  Files         216      216              
  Lines       17860    17860              
==========================================
- Hits         9225     9224       -1     
- Misses       8159     8160       +1     
  Partials      476      476

@thaJeztah thaJeztah left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM ❤️

@vdemeester

Copy link
Copy Markdown
Collaborator Author

@dnephin I'm not sure why e2e failed on that one 🤔

@dnephin

dnephin commented Nov 29, 2017

Copy link
Copy Markdown
Contributor

The test is broken I think, I'll look at fixing it. Somehow the build output changed in 17.11 I think.

@dnephin dnephin 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

@dnephin
dnephin merged commit 8abb266 into docker:master Nov 29, 2017
@GordonTheTurtle GordonTheTurtle added this to the 17.12.0 milestone Nov 29, 2017
@vdemeester
vdemeester deleted the bump-go branch November 29, 2017 14:46
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