-
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
Add go 1.19, require go 1.18, drop go 1.17 #3514
Conversation
8e2e2d7
to
34e2375
Compare
OK, it seems to be working fine, except for golangci-lint (filed golangci/golangci-lint#2922) |
1.19 is GA now |
Alas we have to wait for a new golangci-lint release that has golangci/golangci-lint#2922 fixed. The fix is already merged but no release has been made yet (I guess it will be v1.48). |
8e8aefe
to
f2116d5
Compare
Since I've already marked 1.19.x CI tests as required, can we please merge this? @opencontainers/runc-maintainers PTAL |
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
Since Go 1.19, godoc recognizes lists, code blocks, headings etc. It also reformats the sources making it more apparent that these features are used. Fix a few places where it misinterpreted the formatting (such as indented vs unindented), and format the result using the gofumpt from HEAD, which already incorporates gofmt 1.19 changes. Some more fixes (and enhancements) might be required. Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
This version works with go 1.19, i.e. it fixes golangci/golangci-lint#2922. Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
@AkihiroSuda PTAL 🙏🏻 |
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
@kolyshkin are you updating the repo settings? ("add test (1.19.x) to required list of tests (once this is merged)") |
This makes go 1.18 a minimally required version, removing go 1.17 from and adding go 1.19 to CI.
Also:
+build
tagstest (1.17.x)
from the required set of teststest (1.19.x)
to required list of tests (once this is merged)