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

golangci-lint: remove Dockerfile.golangci-lint and update renovate config #2044

Merged
merged 3 commits into from
Jan 31, 2024

Conversation

mtardy
Copy link
Member

@mtardy mtardy commented Jan 31, 2024

See commits.

@mtardy mtardy added the release-note/dependency This PR updates one or multiple dependencies label Jan 31, 2024
@mtardy mtardy requested a review from a team as a code owner January 31, 2024 11:39
@mtardy mtardy requested a review from olsajiri January 31, 2024 11:39
We no longer need the special golangci-lint docker image since we now
build without CGO and can use the upstream image directly.

Secondly, this adds GOTOOLCHAIN=auto to the golangci-lint container
since it might ship older Go toolchain than our go.mod directive and
thus not being able to build the project.

Signed-off-by: Mahe Tardy <[email protected]>
This was removed in favor of using the full docker image reference in
previous commits.

Signed-off-by: Mahe Tardy <[email protected]>
Instead of focusing only on upgrading a specific line starting with
`GO_IMAGE`, the new regex allow it to work with any variable name.

Signed-off-by: Mahe Tardy <[email protected]>
@mtardy mtardy force-pushed the pr/mtardy/renovate-regex-makefile branch from 64dca44 to b6428a1 Compare January 31, 2024 12:10
@mtardy mtardy requested review from kkourt and removed request for olsajiri January 31, 2024 12:11
@mtardy mtardy changed the title renovate: make Makefile docker image regex more general golangci-lint: remove Dockerfile.golangci-lint and update renovate config Jan 31, 2024
@mtardy mtardy merged commit 24e3421 into main Jan 31, 2024
35 checks passed
@mtardy mtardy deleted the pr/mtardy/renovate-regex-makefile branch January 31, 2024 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/dependency This PR updates one or multiple dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants