Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

chore: move tools to a separated module #816

Merged
merged 1 commit into from
Jul 24, 2024
Merged

Conversation

janisz
Copy link
Collaborator

@janisz janisz commented Jul 18, 2024

External tools pollute go.mod and make it harder to update project. In this PR new module is created for golangci-lint and goreleaser and Go is updated to 1.22.5

Signed-off-by: Tomasz Janiszewski <tomek@redhat.com>
@janisz janisz force-pushed the move_tools_to_a_new_module branch from c753769 to 99490d8 Compare July 18, 2024 15:29
@@ -1,5 +1,7 @@
.PHONY: deps
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This was a mistake we create a file here

@@ -10,5 +10,4 @@ package toolimports
import (
_ "github.com/golangci/golangci-lint/cmd/golangci-lint"
_ "github.com/goreleaser/goreleaser"
_ "k8s.io/cli-runtime/pkg/resource" // This is imported because we want to be able to require it at a certain version, since otherwise Helm breaks.
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

let's see if we really need it

@janisz janisz merged commit 62e519c into main Jul 24, 2024
8 checks passed
@janisz janisz deleted the move_tools_to_a_new_module branch July 24, 2024 08:53
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants