-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
update to go1.13 #1376
update to go1.13 #1376
Conversation
Signed-off-by: Zach Badgett <[email protected]>
Signed-off-by: Zach Badgett <[email protected]>
Signed-off-by: Tonis Tiigi <[email protected]>
91cdf5c
to
a60ecfa
Compare
@@ -79,3 +79,7 @@ require ( | |||
replace github.com/hashicorp/go-immutable-radix => github.com/tonistiigi/go-immutable-radix v0.0.0-20170803185627-826af9ccf0fe | |||
|
|||
replace github.com/jaguilar/vt100 => github.com/tonistiigi/vt100 v0.0.0-20190402012908-ad4c4a574305 | |||
|
|||
replace github.com/containerd/containerd => github.com/containerd/containerd v1.3.1-0.20191014053712-acdcf13d5eaf |
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.
github.com/containerd/containerd v1.4.0-0.20191014053712-acdcf13d5eaf
in L13 should be like github.com/containerd/containerd v0.0.0
?
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.
Is there a case that requires this. As long as replace has the "sanitized" version it seems to work for me.
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.
Just for human understandability
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.
v1.4.0-0
might improve understandability as well, as it is "newer than 1.3"
carry #1297