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