Merge #1509

1509: Update golangci, trying to fix lint r=Kay-Zee a=Kay-Zee This updates the golangci action to just use the latest v2 action, which is recommended for the action. Also, skipping the PKG cache because of all the File Exists error. Will have to follow: golangci/golangci-lint-action#244 to see when this is resolved. (though lint is one of the faster jobs, so it's fine even if we skip the cache) Lastly, I'm suggesting we skip the `unused` lint for TEST FILES ONLY, as there are many cases where we want to use `t.Skip()` to quarantine, but then it causes "unused" errors. This will resolve those cases and better allow us to use `t.Skip()` Co-authored-by: Kay-Zee <[email protected]>
onflow · Oct 22, 2021 · ac73802 · ac73802
2 parents 11299bb + 8503a6a
commit ac73802
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -26,11 +26,12 @@ jobs:
     - name: Build relic
       run: make crypto/relic/build
     - name: Run golangci-lint
-      uses: golangci/golangci-lint-action@v2.3.0
+      uses: golangci/golangci-lint-action@v2
       with:
         # Required: the version of golangci-lint is required and must be specified without patch version: we always use the latest patch version.
         version: v1.41
         args: -v --build-tags relic
+        skip-pkg-cache: true
     - name: Run ShellCheck
       uses: ludeeus/action-shellcheck@master
       with:

diff --git a/.golangci.yml b/.golangci.yml
@@ -22,3 +22,9 @@ linters:
   enable:
     - goimports
     - gosec
+
+issues:
+  exclude-rules:
+    - path: _test\.go # disable some linters on test files
+      linters:
+        - unused