ci: add lint job to test workflow

Broderick-Westrope · Aug 18, 2024 · 37c96d2 · 37c96d2
1 parent e6ac7f8
commit 37c96d2
Showing 1 changed file with 24 additions and 4 deletions.
diff --git a/.github/workflows/test.yaml b/.github/workflows/test.yaml
@@ -13,9 +13,9 @@ jobs:
     name: Test
     strategy:
       matrix:
-        go-version: [1.22.x, 1.23.x]
-        platform: [ubuntu-latest, macos-latest, windows-latest]
-    runs-on: ${{matrix.platform}}
+        go: [1.22.x, 1.23.x]
+        os: [ubuntu-latest, macos-latest, windows-latest]
+    runs-on: ${{matrix.os}}
     steps:
       - name: Set up Go ${{matrix.go-version}}
         uses: actions/setup-go@v5
@@ -32,5 +32,25 @@ jobs:
       - name: Build
         run: go build -o ./bin/tetrigo -v ./cmd/tetrigo
 
+      - name: Lint
+        run: golangci-lint run
+
       - name: Test
-        run: go test -cover -count=1 -race ./...
+        run: go test -cover -count=1 -race ./...
+
+  lint:
+    name: Lint
+    strategy:
+      matrix:
+        go: [1.23.x]
+        os: [ubuntu-latest, macos-latest, windows-latest]
+    runs-on: ${{ matrix.os }}
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-go@v5
+        with:
+          go-version: ${{ matrix.go }}
+      - name: golangci-lint
+        uses: golangci/golangci-lint-action@v6
+        with:
+          version: v1.59