Update version, remove deprecated linter (#265)

helsaawy · web-flow · commit 650a2e464c74 · 2022-10-28T12:31:56.000-04:00
`structcheck` is deprecated.
Update golanci-lint to 1.50.

Signed-off-by: Hamza El-Saawy &lt;hamzaelsaawy@microsoft.com&gt;
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -18,7 +18,7 @@ jobs:
       - name: Run golangci-lint
         uses: golangci/golangci-lint-action@v3
         with:
-          version: v1.48
+          version: v1.50
           args: >-
             --verbose
             --timeout=5m
diff --git a/.golangci.yml b/.golangci.yml
@@ -20,7 +20,6 @@ linters:
     - gofmt # files are gofmt'ed
     - gosec # security
     - nilerr # returns nil even with non-nil error
-    - structcheck # unused struct fields
     - unparam # unused function params
 
 issues:
@@ -94,6 +93,8 @@ linters-settings:
         disabled: true
       - name: flag-parameter # excessive, and a common idiom we use
         disabled: true
+      - name: unhandled-error # warns over common fmt.Print* and io.Close; rely on errcheck instead
+        disabled: true
       # general config
       - name: line-length-limit
         arguments:
