Introduce additional linters & bump golangci-lint to 1.50.1

Signed-off-by: Prajyot-Parab <[email protected]>

.github/workflows/golangci-lint.yml

@@ -25,5 +25,5 @@ jobs:
       - name: golangci-lint
         uses: golangci/golangci-lint-action@0ad9a0988b3973e851ab0a07adf248ec2e100376 # tag=v3.3.1
         with:
-          version: v1.50.0
+          version: v1.50.1
           working-directory: ${{matrix.working-directory}}

.golangci.yml

@@ -17,12 +17,14 @@ linters:
   - bidichk
   - bodyclose
   - containedctx
+  - decorder
   - depguard
   - dogsled
   - dupword
   - durationcheck
   - errcheck
   - errchkjson
+  - execinquery
   - exportloopref
   - gci
   - goconst
@@ -41,8 +43,10 @@ linters:
   - nilerr
   - noctx
   - nolintlint
+  - nosprintfhostport
   - prealloc
   - predeclared
+  - reassign
   - revive
   - rowserrcheck
   - staticcheck
@@ -52,6 +56,7 @@ linters:
   - unconvert
   - unparam
   - unused
+  - usestdlibvars
   - whitespace
 
 linters-settings:

cmd/clusterctl/client/repository/goproxy.go

@@ -74,7 +74,7 @@ func (g *goproxyClient) getVersions(ctx context.Context, base, owner, repository
 		}
 		defer resp.Body.Close()
 
-		if resp.StatusCode != 200 {
+		if resp.StatusCode != http.StatusOK {
 			retryError = errors.Errorf("failed to get versions: response status code %d", resp.StatusCode)
 			return false, nil
 		}