linter updates (#150)

Author: soumeh01

.github/workflows/test.yml

@@ -62,7 +62,7 @@ jobs:
           check-latest: true
 
       - name: Lint with golangci-lint
-        uses: golangci/golangci-lint-action@4696ba8babb6127d732c3c6dde519db15edab9ea # v6.5.1
+        uses: golangci/golangci-lint-action@1481404843c368bc19ca9406f87d6e0fc97bdcfd # v7.0.0
         with:
           version: latest
 

.golangci.yml

@@ -1,29 +1,30 @@
+version: "2"
 run:
-  timeout: 10m
-
-  # Run linters over integration tests
   build-tags:
     - integration
-
 linters:
-  disable-all: true # Disable defaults, then enable the ones we want
+  default: none
   enable:
     - errcheck
-    - gosimple
+    - gosec
     - govet
     - ineffassign
     - staticcheck
-    # - structcheck https://github.com/golangci/golangci-lint/issues/2649
-    - typecheck
     - unused
-    # - bodyclose Disabled so concurrent GETs can happen
-    - stylecheck
-    - gosec
-    - goimports
+  settings:
+    staticcheck:
+      checks:
+        - -SA1019
+        - all
+  exclusions:
+    generated: lax
+    presets:
+      - comments
+      - common-false-positives
+      - legacy
+      - std-error-handling
+formatters:
+  enable:
     - gci
-
-linters-settings:
-  staticcheck:
-    checks:
-      - all
-      - "-SA1019" # Disable "rand.Seed has been deprecated"
+    - goimports
+    - gofmt

internal/stm32CubeMX/stm32CubeMX.go

@@ -215,6 +215,7 @@ func Process(cbuildGenIdxYmlPath, outPath, cubeMxPath string, runCubeMx bool, pi
 					}
 					fIoc.Close()
 					for { // wait for .mxproject change
+						//nolint:staticcheck // intentional logic for clarity
 						if !running {
 							break // out of loop if CubeMX does not run anymore
 						}