-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Bump Go to 1.9.3 #827
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bump Go to 1.9.3 #827
Conversation
|
Needs #826 to fix failures https://hub.docker.com/r/dockercore/golang-cross/builds/bffpend2ypnea9bbl6t2acj/ |
30227ed to
3d25831
Compare
Codecov Report
@@ Coverage Diff @@
## master #827 +/- ##
=======================================
Coverage 52.96% 52.96%
=======================================
Files 244 244
Lines 15828 15828
=======================================
Hits 8383 8383
Misses 6891 6891
Partials 554 554 |
vdemeester
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🐻
|
don't merge yet, I have to update https://hub.docker.com/r/dockercore/golang-cross/tags/ as well |
|
Just tagged
|
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
3d25831 to
ffc7648
Compare
|
Thanks! Updated 👍 |
|
Thanks! |
Bump Go to 1.9.3 Upstream-commit: 819df0e Component: cli
still needs docker/golang-cross#3 to be merged, and a new version built on Docker Hub so that we can update the Dockerfile.cross
release notes: https://golang.org/doc/devel/release.html#go1.9.minor
full diff: golang/go@go1.9.2...go1.9.3
ping @vdemeester @dnephin