-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
ci: add go1.17 #3021
ci: add go1.17 #3021
Conversation
68005a2
to
1020824
Compare
02ad38d
to
194b806
Compare
CI failure is unrelated (#3050) |
Rebased. |
Updated to 1.17-rc1 |
Removed go 1.15 |
8b42a04
to
7a0053c
Compare
af5fae9
to
6db56cf
Compare
We have 1.17 GA now https://golang.org/dl/ |
6db56cf
to
daef138
Compare
Updated to GA; rebased; no longer a draft. |
@@ -17,7 +17,7 @@ jobs: | |||
strategy: | |||
fail-fast: false | |||
matrix: | |||
go-version: [1.15.x, 1.16.x] | |||
go-version: [1.15.x, 1.16.x, 1.17.x] |
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.
Dockerfile needs to be update
Line 1 in 55e93b8
ARG GO_VERSION=1.16 |
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.
Can we drop support for Go 1.15?
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.
Dockerfile needs to be update
done
Can we drop support for Go 1.15?
I'd say let's have it for some time as it just became unsupported a day ago.
This is also a separate issue; I'd like to add 1.17 now and drop 1.15 later and separately.
Signed-off-by: Kir Kolyshkin <[email protected]>
daef138
to
a587180
Compare
Keeping this as draft until go1.17 is released. Will rebase periodically to test master with go 1.17.
1.0 backport: #3299