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

[v23.2.x] rpk: manually update golangci-lint version #15364

Merged
merged 1 commit into from
Dec 7, 2023

Conversation

r-vasquez
Copy link
Contributor

@r-vasquez r-vasquez commented Dec 7, 2023

We use latest in dev, but v22.3 was using an old
version that was causing errors in CI

Why?
Our GH actions:

  • Download Go's stable version.
  • Download the pinned version of golangci-lint. (Previously 1.52)

From the GH Actions Doc, Go's stable version comes from the go-version repository manifest and this was updated yesterday to 1.21.5, and golangci-lint@1.52 is incompatible with this new version.

Backports Required

  • none - not a bug fix
  • none - this is a backport
  • none - issue does not exist in previous branches
  • none - papercut/not impactful enough to backport
  • v23.2.x
  • v23.1.x
  • v22.3.x

Release Notes

  • none

We use latest in dev, but v22.3 was using an old
version that was causing errors in CI
@r-vasquez r-vasquez marked this pull request as ready for review December 7, 2023 16:47
@r-vasquez r-vasquez requested review from twmb, gene-redpanda and a team as code owners December 7, 2023 16:47
@r-vasquez r-vasquez requested review from nk-87 and removed request for a team December 7, 2023 16:47
@andrewhsu andrewhsu changed the title rpk: manually update golangci-lint version [v23.3.x] rpk: manually update golangci-lint version Dec 7, 2023
@andrewhsu andrewhsu changed the title [v23.3.x] rpk: manually update golangci-lint version [v23.2.x] rpk: manually update golangci-lint version Dec 7, 2023
Copy link
Member

@andrewhsu andrewhsu left a comment

Choose a reason for hiding this comment

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

LGTM

@r-vasquez r-vasquez merged commit 5079555 into redpanda-data:v23.2.x Dec 7, 2023
33 checks passed
@r-vasquez r-vasquez deleted the update-golangci-lint branch February 1, 2024 22:52
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.

None yet

4 participants