Merge pull request #45 from saschagrunert/lint

Update golangci-lint and fix lints

.golangci.yml

@@ -1,7 +1,7 @@
 ---
 run:
   concurrency: 6
-  deadline: 5m
+  timeout: 5m
 issues:
 linters:
   disable-all: true
@@ -10,9 +10,11 @@ linters:
     - asciicheck
     - bidichk
     - bodyclose
+    - canonicalheader
     - containedctx
     - contextcheck
     - copyloopvar
+    - cyclop
     - decorder
     - dogsled
     - dupl
@@ -22,9 +24,9 @@ linters:
     - errchkjson
     - errname
     - errorlint
-    - execinquery
     - exhaustive
     - exportloopref
+    - fatcontext
     - forcetypeassert
     - funlen
     - gci
@@ -59,6 +61,7 @@ linters:
     - makezero
     - mirror
     - misspell
+    - mnd
     - musttag
     - nakedret
     - nestif
@@ -97,14 +100,12 @@ linters:
     - whitespace
     - wrapcheck
     - zerologlint
-    # - cyclop
     # - depguard
+    # - err113
     # - exhaustruct
     # - forbidigo
     # - gochecknoglobals
     # - gochecknoinits
-    # - goerr113
-    # - gomnd
     # - ireturn
     # - lll
     # - nlreturn
@@ -113,6 +114,8 @@ linters:
     # - varnamelen
     # - wsl
 linters-settings:
+  cyclop:
+    max-complexity: 15
   godox:
     keywords:
       - BUG
@@ -122,75 +125,4 @@ linters-settings:
     check-type-assertions: true
     check-blank: true
   gocritic:
-    enabled-checks:
-      - appendCombine
-      - badLock
-      - badRegexp
-      - badSorting
-      - badSyncOnceFunc
-      - boolExprSimplify
-      - builtinShadow
-      - builtinShadowDecl
-      - commentedOutCode
-      - commentedOutImport
-      - deferInLoop
-      - deferUnlambda
-      - docStub
-      - dupImport
-      - dynamicFmtString
-      - emptyDecl
-      - emptyFallthrough
-      - emptyStringTest
-      - equalFold
-      - evalOrder
-      - exposedSyncMutex
-      - externalErrorReassign
-      - filepathJoin
-      - hexLiteral
-      - httpNoBody
-      - hugeParam
-      - importShadow
-      - indexAlloc
-      - initClause
-      - methodExprCall
-      - nestingReduce
-      - nilValReturn
-      - octalLiteral
-      - paramTypeCombine
-      - preferDecodeRune
-      - preferFilepathJoin
-      - preferFprint
-      - preferStringWriter
-      - preferWriteByte
-      - ptrToRefParam
-      - rangeExprCopy
-      - rangeValCopy
-      - redundantSprint
-      - regexpPattern
-      - regexpSimplify
-      - returnAfterHttpError
-      - ruleguard
-      - sliceClear
-      - sloppyReassign
-      - sortSlice
-      - sprintfQuotedString
-      - sqlQuery
-      - stringConcatSimplify
-      - stringXbytes
-      - stringsCompare
-      - syncMapLoadAndDelete
-      - timeExprSimplify
-      - todoCommentWithoutDetail
-      - tooManyResultsChecker
-      - truncateCmp
-      - typeAssertChain
-      - typeDefFirst
-      - typeUnparen
-      - uncheckedInlineErr
-      - unlabelStmt
-      - unnamedResult
-      - unnecessaryBlock
-      - unnecessaryDefer
-      - weakCond
-      - whyNoLint
-      - yodaStyleExpr
+    enable-all: true

go.mod

@@ -1,6 +1,6 @@
 module sigs.k8s.io/mdtoc
 
-go 1.21
+go 1.22
 
 require (
 	github.com/gomarkdown/markdown v0.0.0-20240328165702-4d01890c35c0

hack/verify-golangci-lint.sh

@@ -18,7 +18,7 @@ set -o errexit
 set -o nounset
 set -o pipefail
 
-VERSION=v1.57.1
+VERSION=v1.59.1
 URL_BASE=https://raw.githubusercontent.com/golangci/golangci-lint
 URL=$URL_BASE/$VERSION/install.sh
 

mdtoc_test.go

@@ -84,7 +84,6 @@ func testdata(subpath string) string {
 func TestDryRun(t *testing.T) {
 	t.Parallel()
 	for _, test := range testcases {
-		test := test
 		t.Run(test.file, func(t *testing.T) {
 			t.Parallel()
 			opts := utilityOptions{
@@ -111,7 +110,6 @@ func TestDryRun(t *testing.T) {
 func TestInplace(t *testing.T) {
 	t.Parallel()
 	for _, test := range testcases {
-		test := test
 		t.Run(test.file, func(t *testing.T) {
 			t.Parallel()
 			original, err := os.ReadFile(test.file)
@@ -157,7 +155,6 @@ func TestInplace(t *testing.T) {
 func TestOutput(t *testing.T) {
 	t.Parallel()
 	for _, test := range testcases {
-		test := test
 		// Ignore the invalid cases, they're only for inplace tests.
 		if !test.validTOCTags {
 			continue

pkg/mdtoc/mdtoc.go

@@ -256,7 +256,8 @@ func GetTOC(file string, opts Options) (string, error) {
 // A temporary file is used so no changes are made to the original in the case of an error.
 func atomicWrite(filePath string, chunks ...string) error {
 	tmpPath := filePath + "_tmp"
-	tmp, err := os.OpenFile(tmpPath, os.O_WRONLY|os.O_CREATE|os.O_EXCL, 0o600)
+	const perms = 0o600
+	tmp, err := os.OpenFile(tmpPath, os.O_WRONLY|os.O_CREATE|os.O_EXCL, perms)
 	if err != nil {
 		return fmt.Errorf("unable to open tepmorary file %s: %w", tmpPath, err)
 	}