chore: introduce linter (#53)

.github/workflows/golangci-lint.yaml

@@ -0,0 +1,18 @@
+name: golangci-lint
+on:
+  pull_request:
+permissions:
+  contents: read
+jobs:
+  golangci:
+    name: lint
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/setup-go@v3
+        with:
+          go-version: 1.18
+      - uses: actions/checkout@v3
+      - name: golangci-lint
+        uses: golangci/golangci-lint-action@v3
+        with:
+          args: --timeout=5m

actions/check_node_status_test.go

@@ -2,11 +2,12 @@ package actions
 
 import (
 	"context"
-	"github.com/google/uuid"
 	"sync"
 	"testing"
 	"time"
 
+	"github.com/google/uuid"
+
 	"github.com/sirupsen/logrus"
 	"github.com/stretchr/testify/require"
 	v1 "k8s.io/api/core/v1"
@@ -150,7 +151,7 @@ func TestCheckStatus_Ready(t *testing.T) {
 		go func() {
 			time.Sleep(1 * time.Second)
 			node.Status.Conditions[0].Status = v1.ConditionTrue
-			clientset.CoreV1().Nodes().Update(context.Background(), node, metav1.UpdateOptions{})
+			_, _ = clientset.CoreV1().Nodes().Update(context.Background(), node, metav1.UpdateOptions{})
 			wg.Done()
 		}()
 		wg.Wait()

castai/client.go

@@ -3,7 +3,6 @@ package castai
 import (
 	"context"
 	"fmt"
-	"net/http"
 	"time"
 
 	"github.com/go-resty/resty/v2"
@@ -38,8 +37,8 @@ func NewDefaultClient(url, key string, level logrus.Level, binVersion *config.Cl
 	client := resty.New()
 	client.SetHostURL(url)
 	client.SetTimeout(5 * time.Minute) // Hard timeout for any request.
-	client.Header.Set(http.CanonicalHeaderKey(headerAPIKey), key)
-	client.Header.Set(http.CanonicalHeaderKey(headerUserAgent), "castai-cluster-controller/"+binVersion.Version)
+	client.Header.Set(headerAPIKey, key)
+	client.Header.Set(headerUserAgent, "castai-cluster-controller/"+binVersion.Version)
 	if level == logrus.TraceLevel {
 		client.SetDebug(true)
 	}