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.

Sign up for GitHub

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

Jump to bottom

Update Go to 1.19 #779

Merged
merged 3 commits into from
Aug 22, 2022
Merged

Update Go to 1.19 #779

merged 3 commits into from
Aug 22, 2022

Conversation

tklauser
Copy link
Member

No description provided.

@tklauser tklauser requested a review from rolinh August 22, 2022 08:33
@tklauser tklauser requested review from a team as code owners August 22, 2022 08:33
@tklauser tklauser requested review from aanm and nebril and removed request for a team August 22, 2022 08:33
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label PR is blocked until the release note is set label Aug 22, 2022
@tklauser tklauser added the release-note/misc This PR makes changes that have no direct user impact. label Aug 22, 2022
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label PR is blocked until the release note is set label Aug 22, 2022
@rolinh
Copy link
Member

rolinh commented Aug 22, 2022

I think we need to first bump golangci-lint to v1.48.0 for Go 1.19 support.

@tklauser
ci, make: update golangci-lint to 1.48.0
6f8a37f 
Signed-off-by: Tobias Klauser <tobias@cilium.io>
@tklauser
print: format with Go 1.19 gofmt
9007022 
Signed-off-by: Tobias Klauser <tobias@cilium.io>
@tklauser
Update Go to 1.19
85865bd 
Signed-off-by: Tobias Klauser <tobias@cilium.io>
@tklauser tklauser requested a review from a team as a code owner August 22, 2022 08:43
@tklauser tklauser changed the title Update Go to 1.19.0 Update Go to 1.19 Aug 22, 2022
@tklauser tklauser merged commit 0e85e38 into master Aug 22, 2022
@tklauser tklauser deleted the pr/tklauser/update-go-1.19 branch August 22, 2022 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rolinh rolinh rolinh approved these changes

@aanm aanm Awaiting requested review from aanm aanm was automatically assigned from cilium/github-sec

@nebril nebril Awaiting requested review from nebril nebril was automatically assigned from cilium/ci-structure

Assignees
No one assigned
Labels
release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@tklauser @rolinh