ci: add golangci-lint workflow

.github/workflows/golangci-lint.yml

@@ -0,0 +1,31 @@
+name: golangci-lint
+on: pull_request
+env:
+  GOLANGCI_LINT_VERSION: "v1.50.1"
+  GO_VERSION: "1.19"
+jobs:
+  golangci-lint:
+    name: golangci-lint
+    runs-on: ubuntu-22.04
+    strategy:
+      fail-fast: false
+      matrix:
+        config:
+          - name: "keptn-lifecycle-operator"
+            folder: "operator/"
+          - name: "scheduler"
+            folder: "scheduler/"
+    steps:
+      - name: Check out code into the Go module directory
+        uses: actions/checkout@v3
+
+      - uses: actions/setup-go@v3
+        with:
+          go-version: ${{ env.GO_VERSION }}
+
+      - name: golangci-lint
+        uses: golangci/golangci-lint-action@v3
+        with:
+          working-directory: ${{ matrix.config.folder }}
+          version: ${{ env.GOLANGCI_LINT_VERSION }}
+          args: --config ../.golangci.yml -v

.golangci.yml

@@ -0,0 +1,12 @@
+run:
+  timeout: 5m
+  go: '1.19'
+linters:
+  enable:
+    - gofmt         # Gofmt checks whether code was gofmt-ed. By default this tool runs with -s option to check for code simplification
+#    - dogsled       # Checks assignments with too many blank identifiers (e.g. x, , , _, := f())
+#    - containedctx  # containedctx is a linter that detects struct contained context.Context field
+#    - errorlint     # errorlint is a linter for that can be used to find code that will cause problems with the error wrapping scheme introduced in Go 1.13.
+#    - gci           # Gci controls golang package import order and makes it always deterministic.
+#    - nilnil        # Checks that there is no simultaneous return of nil error and an invalid value.
+#    - noctx         # noctx finds sending http request without context.Context

operator/controllers/common/activemetricsobject.go

@@ -5,8 +5,9 @@ import (
 	"sigs.k8s.io/controller-runtime/pkg/client"
 )
 
+// ActiveMetricsObject represents an object whose active metrics are stored
+//
 //go:generate moq -pkg fake --skip-ensure -out ./fake/activemetricsobject_mock.go . ActiveMetricsObject
-//ActiveMetricsObject represents an object whose active metrics are stored
 type ActiveMetricsObject interface {
 	GetActiveMetricsAttributes() []attribute.KeyValue
 	IsEndTimeSet() bool