golangci: use 'revive' instead of 'golint'

spf13 · Jul 17, 2023 · cccae8e · cccae8e
1 parent 1916426
commit cccae8e
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/.golangci.yml b/.golangci.yml
@@ -68,7 +68,7 @@ linters:
     #- gofumpt
     #- goheader
     - goimports
-    - golint # !
+    #- golint ! Use revive instead
     #- gomnd
     #- gomoddirectives
     #- gomodguard
@@ -99,7 +99,7 @@ linters:
     #- prealloc
     #- predeclared
     #- promlinter
-    #- revive
+    - revive
     #- rowserrcheck
     #- scopelint !
     #- sqlclosecheck

diff --git a/completions_test.go b/completions_test.go
@@ -1261,7 +1261,7 @@ func TestValidArgsFuncCmdContext(t *testing.T) {
 	}
 	rootCmd.AddCommand(childCmd)
 
-	//nolint:golint,staticcheck // We can safely use a basic type as key in tests.
+	//nolint:golint,revive,staticcheck // We can safely use a basic type as key in tests.
 	ctx := context.WithValue(context.Background(), "testKey", "123")
 
 	// Test completing an empty string on the childCmd