Skip to content

Upgrade go to 1.20 #320

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

Merged
merged 1 commit into from
Mar 17, 2023
Merged

Upgrade go to 1.20 #320

merged 1 commit into from
Mar 17, 2023

Conversation

lennysgarage
Copy link
Member

Signed-off-by: Jonathan Marcantonio [email protected]

Updates golangci-lint from v1.46.2 to v1.51.2
As well upgrades these k8s apis to their latest patch version from v0.25.2 to v0.25.7

        k8s.io/api
	k8s.io/apiextensions-apiserver 
	k8s.io/apimachinery 
	k8s.io/client-go 

An additional space has been added to many comments due to an update with the gofmt command in 1.20. As well mentions of //nolint needed to be changed to //nolint:all, golang/vscode-go#2477 (comment).

Addresses:

@openshift-ci openshift-ci bot requested review from jnpacker and xiangjingli March 16, 2023 16:44
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Mar 16, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: lennysgarage

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@lennysgarage lennysgarage requested review from chenz4027 and removed request for jnpacker March 16, 2023 16:45
@lennysgarage lennysgarage force-pushed the upgrade-go-20 branch 4 times, most recently from 85497c7 to cfd42b3 Compare March 16, 2023 20:54
@xiangjingli
Copy link
Collaborator

@lennysgarage @chenz4027 I would suggest to update channel repo first. Once it is done, pick up the latest version of channel repo in the subscription repo.

Signed-off-by: Jonathan Marcantonio <[email protected]>

Update k8s.io apis to v0.25.7

Signed-off-by: Jonathan Marcantonio <[email protected]>

update client-go

Signed-off-by: Jonathan Marcantonio <[email protected]>

update linter

Signed-off-by: Jonathan Marcantonio <[email protected]>

Update linter again

Signed-off-by: Jonathan Marcantonio <[email protected]>

musttag linter

Signed-off-by: Jonathan Marcantonio <[email protected]>

Never backported this change

Signed-off-by: Jonathan Marcantonio <[email protected]>

Update channel in go.mod

Signed-off-by: Jonathan Marcantonio <[email protected]>
@xiangjingli
Copy link
Collaborator

/lgtm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants