Skip to content

Commit

Permalink
Upgrade golangci to disable loopvar in Go 1.22
Browse files Browse the repository at this point in the history
  • Loading branch information
sethvargo committed Mar 13, 2024
1 parent e8ce6e3 commit 87c6d5c
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 2 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/go-lint.yml
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ on:
golangci_lint_version:
description: 'Version of golangci linter to use'
type: 'string'
default: 'v1.55'
default: 'v1.56'

jobs:
# modules checks if the go modules are all up-to-date. While rare with modern
Expand Down
2 changes: 2 additions & 0 deletions .golangci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -76,8 +76,10 @@ linters:
- 'paralleltest'
- 'prealloc'
- 'predeclared'
- 'protogetter'
- 'rowserrcheck'
- 'sloglint'
- 'spancheck'
- 'sqlclosecheck'
- 'staticcheck'
- 'stylecheck'
Expand Down
1 change: 0 additions & 1 deletion testutil/integ_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@ import (
"testing"
)

//nolint:paralleltest // Can't be paralleled because of t.Setenv
func TestIsIntegration(t *testing.T) {
if IsIntegration(t) {
t.Errorf("IsIntegration() got 'true' want 'false'")
Expand Down

0 comments on commit 87c6d5c

Please sign in to comment.