chore(fix): enables more linters (#54)

Author: guewa

.golangci.yaml

@@ -1,7 +1,29 @@
 version: "2"
 run:
-  concurrency: 4
+  allow-parallel-runners: true
 linters:
+  default: none
+  enable:
+    - copyloopvar
+    - dupl
+    - errcheck
+    - ginkgolinter
+    - goconst
+    - gocyclo
+    - govet
+    - ineffassign
+    - misspell
+    - nakedret
+    - prealloc
+    - revive
+    - staticcheck
+    - unconvert
+#    - unparam
+    - unused
+  settings:
+    revive:
+      rules:
+        - name: comment-spacings
   exclusions:
     generated: lax
     presets:
@@ -16,6 +38,9 @@ linters:
       - builtin$
       - examples$
 formatters:
+  enable:
+    - gofmt
+    - goimports
   exclusions:
     generated: lax
     paths:

internal/ocm/utils.go

@@ -17,7 +17,7 @@ type CatalogResponse struct {
 	Repositories []string `json:"repositories"`
 }
 
-/**
+/*
  * @Description: GetRepositoriesInOCIRegistry returns the repositories in the OCI registry
  * @param ociRegistry string - the OCI registry URL without paths
  * @param creds RegistryCredentials - the credentials to access the OCI registry

pkg/juggler/fluxcd/flux_sources.go

@@ -22,7 +22,7 @@ const (
 	msgReadyNotPresent = "Unable to check healthiness. Ready condition is not present."
 )
 
-/// use strategy for  with adapter pattern
+// use strategy for  with adapter pattern
 // adapter for third party types to conform to a common interface
 // use strategy for the reconcilers on how each can be reconciled e.g. for the applyResources method in flux-reconciler