Split lint, unit test jobs & bump linters (#271)

* Split lint, unit test jobs

There's no reason we need to constrain our version of Go for linters:
It is only necessary for unit tests. Make this possible by splitting our
workflows.

Signed-off-by: Stephen Finucane <[email protected]>

* Bump golangci-lint, gotestsum

Signed-off-by: Stephen Finucane <[email protected]>

---------

Signed-off-by: Stephen Finucane <[email protected]>

Author: stephenfin

.github/workflows/unit.yml renamed to .github/workflows/lint.yaml

@@ -5,23 +5,16 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       fail-fast: false
-      matrix:
-        go-version:
-          - "1.22"
 
     steps:
       - name: Checkout source
         uses: actions/checkout@v5
 
-      - name: Setup Go ${{ matrix.go-version }}
+      - name: Setup Go
         uses: actions/setup-go@v5
         with:
-          go-version: ${{ matrix.go-version }}
+          go-version: "1"
 
       - name: Run linters
         run: |
           make lint
-
-      - name: Run unit tests
-        run: |
-          make unit

.github/workflows/unit.yaml

@@ -0,0 +1,17 @@
+name: Unit Testing
+on: [push, pull_request]
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout source
+        uses: actions/checkout@v5
+
+      - name: Setup Go
+        uses: actions/setup-go@v5
+        with:
+          go-version-file: 'go.mod'
+
+      - name: Run unit tests
+        run: |
+          make unit

Makefile

@@ -1,7 +1,7 @@
 undefine GOFLAGS
 
-GOLANGCI_LINT_VERSION?=v2.1.6
-GOTESTSUM_VERSION?=v1.12.2
+GOLANGCI_LINT_VERSION?=v2.5.0
+GOTESTSUM_VERSION?=v1.13.0
 GO_TEST?=go run gotest.tools/gotestsum@$(GOTESTSUM_VERSION) --format testname --
 TIMEOUT := "60m"