Skip to content
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 to 1.18 #759

Merged
merged 2 commits into from
Mar 17, 2022
Merged

Update Go to 1.18 #759

merged 2 commits into from
Mar 17, 2022

Conversation

tklauser
Copy link
Member

@tklauser tklauser commented Mar 17, 2022

Also bump golangci-lint to v1.44.2

@tklauser tklauser temporarily deployed to ci March 17, 2022 09:47 Inactive
@tklauser tklauser force-pushed the pr/tklauser/update-go-1.18 branch from b0e92a2 to 113b4eb Compare March 17, 2022 09:55
@tklauser tklauser temporarily deployed to ci March 17, 2022 09:55 Inactive
@tklauser tklauser added the dont-merge/waiting-for-upstream Only merge once upstream library or kernel features have landed label Mar 17, 2022
Also drop the GH action skip-go-installation option which was removed in
v3 of the golangci-lint action (see
https://github.com/golangci/golangci-lint-action#compatibility) and
switch to skip-cache as the new catch-all (suggested by Robin).

Signed-off-by: Tobias Klauser <[email protected]>
@tklauser tklauser force-pushed the pr/tklauser/update-go-1.18 branch from 113b4eb to 05138ce Compare March 17, 2022 10:56
@tklauser tklauser temporarily deployed to ci March 17, 2022 10:56 Inactive
@tklauser tklauser removed the dont-merge/waiting-for-upstream Only merge once upstream library or kernel features have landed label Mar 17, 2022
@tklauser tklauser force-pushed the pr/tklauser/update-go-1.18 branch from 05138ce to 65bb536 Compare March 17, 2022 11:05
@tklauser tklauser force-pushed the pr/tklauser/update-go-1.18 branch from 65bb536 to faff75e Compare March 17, 2022 11:07
Switch to run golangci-lint using `make check` in CI for now, using the
golangci-lint GH action with Go 1.18 is not yet fully
supported, see golangci/golangci-lint#2649

This approach was suggested by Robin.

Signed-off-by: Tobias Klauser <[email protected]>
@tklauser tklauser force-pushed the pr/tklauser/update-go-1.18 branch from faff75e to 673e834 Compare March 17, 2022 11:12
@tklauser tklauser temporarily deployed to ci March 17, 2022 11:12 Inactive
@tklauser tklauser marked this pull request as ready for review March 17, 2022 11:18
@tklauser tklauser requested review from a team as code owners March 17, 2022 11:18
@tklauser tklauser merged commit b79a17a into master Mar 17, 2022
@tklauser tklauser deleted the pr/tklauser/update-go-1.18 branch March 17, 2022 17:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants