-
Notifications
You must be signed in to change notification settings - Fork 2k
Bump Golang to 1.10.3 #1122
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 Golang to 1.10.3 #1122
Conversation
|
Waiting for docker/golang-cross#7 to be built on Docker Hub |
1dc1694 to
58146a3
Compare
|
Ok, this is ready now; ping @vdemeester @silvin-lubecki @dnephin PTAL |
go1.10.3 (released 2018/06/05) includes fixes to the go command, and the crypto/tls, crypto/x509, and strings packages. In particular, it adds minimal support to the go command for the vgo transition. See the Go 1.10.3 milestone on our issue tracker for details; https://github.com/golang/go/issues?q=milestone%3AGo1.10.3 Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
58146a3 to
1c71c95
Compare
|
Updated appveyor.yml as well (thanks @chris-crone !) |
|
ping @vdemeester looks like appveyor isn't actually run; it was added in #905, but maybe never configured? |
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
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!
It appears the appveyor.yml is not being used so we should consider removing it in a followup cc @vdemeester
|
Or preferably, wiring it up, I guess #905 |
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 🐯
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
Should we also change
cli/cli/command/system/version_test.go
Line 55 in 4cb3c70
| GoVersion: "go1.10.2", |
|
Don't think the test needs updating, but we could replace it with some fictive version to make it more clear it's just for testing |
|
Don't know if I have time to do so right now, so if it's ok for a follow up 😄 |
+1 on this but in a followup |

Formed in 2009, the Archive Team (not to be confused with the archive.org Archive-It Team) is a rogue archivist collective dedicated to saving copies of rapidly dying or deleted websites for the sake of history and digital heritage. The group is 100% composed of volunteers and interested parties, and has expanded into a large amount of related projects for saving online and digital history.

go1.10.3 (released 2018/06/05) includes fixes to the go command, and the
crypto/tls, crypto/x509, and strings packages. In particular, it adds minimal
support to the go command for the vgo transition. See the Go 1.10.3 milestone
on our issue tracker for details;
https://github.com/golang/go/issues?q=milestone%3AGo1.10.3