Skip to content

Commit ddf3e24

Browse files
Update dependency golangci/golangci-lint to v2.12.2 (#163)
* Update dependency golangci/golangci-lint to v2.12.2 * use go install as the other dependencies to install golang-ci Signed-off-by: Felix Breuer <f.breuer94@gmail.com> * fix linting issues Signed-off-by: Felix Breuer <f.breuer94@gmail.com> --------- Signed-off-by: Felix Breuer <f.breuer94@gmail.com> Co-authored-by: ske-renovate-ce[bot] <163154779+ske-renovate-ce[bot]@users.noreply.github.com> Co-authored-by: Felix Breuer <f.breuer94@gmail.com>
1 parent 4b33ccf commit ddf3e24

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

hack/tools.mk

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ GOIMPORTS_REVISER_VERSION ?= v3.12.6
99
# renovate: datasource=github-releases depName=ko-build/ko
1010
KO_VERSION ?= v0.18.1
1111
# renovate: datasource=github-releases depName=golangci/golangci-lint
12-
GOLANGCI_LINT_VERSION ?= v2.11.4
12+
GOLANGCI_LINT_VERSION ?= v2.12.2
1313

1414
# Tool targets should declare go.mod as a prerequisite, if the tool's version is managed via go modules. This causes
1515
# make to rebuild the tool in the desired version, when go.mod is changed.
@@ -37,4 +37,4 @@ $(KO): $(call tool_version_file,$(KO),$(KO_VERSION))
3737

3838
GOLANGCI_LINT := $(TOOLS_BIN_DIR)/golangci-lint
3939
$(GOLANGCI_LINT): $(call tool_version_file,$(GOLANGCI_LINT),$(GOLANGCI_LINT_VERSION))
40-
curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sh -s -- -b $(TOOLS_BIN_DIR) $(GOLANGCI_LINT_VERSION)
40+
GOBIN=$(abspath $(TOOLS_BIN_DIR)) go install github.com/golangci/golangci-lint/v2/cmd/golangci-lint@$(GOLANGCI_LINT_VERSION)

test/e2e/e2e_suite_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ import (
55
"fmt"
66
"os"
77
"os/exec"
8+
"slices"
89
"testing"
910

1011
. "github.com/onsi/ginkgo/v2"
@@ -89,8 +90,7 @@ var _ = AfterSuite(func(ctx context.Context) {
8990
_, _ = fmt.Fprintf(GinkgoWriter, "MCM uninstall output: %s\n", string(output))
9091

9192
By("cleaning up tracked test resources after MCM removal")
92-
for i := len(testResources) - 1; i >= 0; i-- {
93-
resource := testResources[i]
93+
for _, resource := range slices.Backward(testResources) {
9494
// Wrap each cleanup in a function with GinkgoRecover to prevent one failure
9595
// from stopping cleanup of remaining resources
9696
func(res TestResource) {

0 commit comments

Comments
 (0)