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

gha: update golangci-lint to v1.46, add Go 1.18, remove Go 1.15 #179

Merged
merged 6 commits into from
Jul 19, 2022

Conversation

thaJeztah
Copy link
Member

See individual commits for details

Signed-off-by: Sebastiaan van Stijn <[email protected]>
Signed-off-by: Sebastiaan van Stijn <[email protected]>
- to get new build-tag format
- to account for go1.19 changes in formatting godoc

Signed-off-by: Sebastiaan van Stijn <[email protected]>
Signed-off-by: Sebastiaan van Stijn <[email protected]>
v3 requires Go to be set up manually (whereas previous
versions would install their own version, which may not
match the expected version).

Also removing the "new code only" constraint; let's just
check all code for issues.

Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah
Copy link
Member Author

We'll need to update the list of required checks after this is merged (otherwise CI will continue waiting for them). Test-run in thaJeztah#1

@thaJeztah thaJeztah marked this pull request as ready for review July 19, 2022 09:35
@thaJeztah
Copy link
Member Author

after this is merged

Ah, ok, looks like it's already needed "before" merging

@thaJeztah
Copy link
Member Author

@kolyshkin @rhatdan PTAL

@rhatdan
Copy link
Collaborator

rhatdan commented Jul 19, 2022

LGTM

@rhatdan rhatdan merged commit 85331a8 into opencontainers:main Jul 19, 2022
@thaJeztah
Copy link
Member Author

Thanks! I updated the repo configuration for the new checks 👍

@thaJeztah thaJeztah deleted the update_golangci_lint branch July 19, 2022 19:10
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.

2 participants