Skip to content

Commit 3d367b1

Browse files
authored
Re-enable copyloopvar linter (#3418)
## Changes Add copyloopvar linter to list of enabled linters. Fix found issues. ## Why This was supposed to be enabled #2160 but it seems I just ran it once at that time.
1 parent 6a854a4 commit 3d367b1

File tree

3 files changed

+1
-2
lines changed

3 files changed

+1
-2
lines changed

.golangci.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@ linters:
1515
- unconvert
1616
- unused
1717
- exhaustruct
18+
- copyloopvar
1819
settings:
1920
copyloopvar:
2021
check-alias: true

integration/bundle/spark_jar_test.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,6 @@ func runSparkJarTests(t *testing.T, testCases []sparkJarTestCase, testRunner fun
4040

4141
// Run the tests that can run
4242
for _, tc := range testCases {
43-
tc := tc // Capture range variable for goroutine
4443
canRun := testCanRun[tc.name]
4544

4645
t.Run(tc.name, func(t *testing.T) {

libs/dagrun/dagrun_test.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,6 @@ func TestRun_VariousGraphsAndPools(t *testing.T) {
7777
}
7878

7979
for _, tc := range tests {
80-
tc := tc
8180
for _, p := range pools {
8281
t.Run(tc.name+fmt.Sprintf(" pool=%d", p), func(t *testing.T) {
8382
g := NewGraph[stringWrapper]()

0 commit comments

Comments
 (0)