bump varnamelen from v0.4.0 to v0.5.0 (#2369)

Author: blizzy78

.golangci.example.yml

@@ -717,13 +717,15 @@ linters-settings:
     ignore-names:
       - err
     # Optional list of variable declarations that should be ignored completely. (defaults to empty list)
-    # Entries must be in the form of "<variable name> <type>" or "<variable name> *<type>".
+    # Entries must be in the form of "<variable name> <type>" or "<variable name> *<type>" for
+    # variables, or "const <name>" for constants.
     ignore-decls:
       - c echo.Context
       - t testing.T
       - f *foo.Bar
       - e error
       - i int
+      - const C
 
   whitespace:
     multi-if: false   # Enforces newlines (or comments) after every multi-line if statement

go.mod

@@ -13,7 +13,7 @@ require (
 	github.com/ashanbrown/forbidigo v1.2.0
 	github.com/ashanbrown/makezero v0.0.0-20210520155254-b6261585ddde
 	github.com/bkielbasa/cyclop v1.2.0
-	github.com/blizzy78/varnamelen v0.4.0
+	github.com/blizzy78/varnamelen v0.5.0
 	github.com/bombsimon/wsl/v3 v3.3.0
 	github.com/breml/bidichk v0.2.1
 	github.com/butuzov/ireturn v0.1.1

go.sum

@@ -121,7 +121,7 @@ func configureCheckerInfo(
 // Maybe in the future, this kind of conversion will be done in go-critic itself.
 func normalizeCheckerParamsValue(lintCtx *linter.Context, p interface{}) interface{} {
 	rv := reflect.ValueOf(p)
-	//nolint:exhaustive // only 3 types (int, bool, and string) are supported by CheckerParam.Value
+	// nolint:exhaustive // only 3 types (int, bool, and string) are supported by CheckerParam.Value
 	switch rv.Type().Kind() {
 	case reflect.Int64, reflect.Int32, reflect.Int16, reflect.Int8, reflect.Int:
 		return int(rv.Int())

pkg/golinters/gocritic.go

@@ -36,7 +36,7 @@ func NewGodot() *goanalysis.Linter {
 		}
 
 		// Convert deprecated setting
-		if cfg.CheckAll { // nolint: staticcheck
+		if cfg.CheckAll { // nolint:staticcheck
 			settings.Scope = godot.TopLevelScope
 		}