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 go.mod to have go1.17 as the minmium version #289
Conversation
manugupt1
commented
Feb 27, 2022
- The Dockerfile already builds for go1.17. Therefore updating the go.mod to go1.17.
- Update README instructions to use go install instead of go get as go get won't add executable binaries in the future.
73146c4
to
524fce6
Compare
.github/workflows/main.yaml
Outdated
@@ -93,8 +93,7 @@ jobs: | |||
- name: "Check out" | |||
uses: actions/checkout@v2 | |||
- name: "Build integration test image" | |||
# Docker 20.10.x builds RootlessKit with Go 1.16 (as of Docker 20.10.11), so we use Go 1.16 as well here. |
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.
This is still true
https://github.com/moby/moby/blob/v20.10.12/Dockerfile#L6
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.
Oh thanks! I can close this PR or are there bits that seem okay to you and I should keep
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.
Docker 20.10.15 will use Go 1.17, so we can drop 1.16 (and add 1.17 instead)
LGTM now, but could you rebase? |
Update go.mod to specify go1.17 as the minimum go version. Also run go mod tidy on it. Signed-off-by: Manu Gupta <manugupt1@gmail.com>
Signed-off-by: Manu Gupta <manugupt1@gmail.com>
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.
Thanks