Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add Go 1.20 support #111

Merged
merged 1 commit into from
Feb 11, 2023
Merged

Conversation

austinvazquez
Copy link
Member

@austinvazquez austinvazquez commented Feb 9, 2023

Updates containerd/project-checks to v1.1.0 to run with Go 1.20. Updates
golangci/golangci-lint to v1.51.1 for Go 1.20 support. Adds Go 1.20 to
lint and test steps. Bumps minimum required version to Go 1.19

Signed-off-by: Austin Vazquez macedonv@amazon.com

@@ -44,7 +44,7 @@ jobs:

strategy:
matrix:
go-version: [1.17.x]
go-version: [1.17.x, 1.18.x, 1.19.x, 1.20.x]
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Linting with each version of Go might be a little pedantic. Lmk your thoughts.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No need to support 1.17 and 1.18

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah, let's just keep it to the last 2 supported releases of Go

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds good. Pushed an update with the changes.

@austinvazquez austinvazquez force-pushed the add-go-1.20-support branch 2 times, most recently from b2ef584 to 0083665 Compare February 9, 2023 19:11
@austinvazquez austinvazquez marked this pull request as ready for review February 9, 2023 19:12
Updates containerd/project-checks to v1.1.0 to run with Go 1.20. Updates
golangci/golangci-lint to v1.51.1 for Go 1.20 support. Adds Go 1.20 to
lint and test steps. Bumps minimum required version to Go 1.19

Signed-off-by: Austin Vazquez <macedonv@amazon.com>
Copy link
Member

@estesp estesp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@AkihiroSuda AkihiroSuda merged commit 6603d5b into containerd:main Feb 11, 2023
@austinvazquez austinvazquez deleted the add-go-1.20-support branch February 23, 2023 18:32
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants