ci: lint and commitlint in its own workflow

Fixes #311
karma-runner · Apr 4, 2022 · 394ba52 · 394ba52
1 parent f78bbec
commit 394ba52
Show file tree

Hide file tree

Showing 2 changed files with 35 additions and 6 deletions.
diff --git a/.github/workflows/lint.yml b/.github/workflows/lint.yml
@@ -0,0 +1,33 @@
+name: Lint
+
+on:
+  pull_request:
+    branches:
+      - master
+
+jobs:
+  linux:
+    name: "Node ${{ matrix.node }} on Linux: Lint"
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        node:
+          - 10
+          - 12
+          - 14
+          - 16
+    steps:
+      - uses: actions/checkout@v3
+        with:
+          fetch-depth: 0
+      - uses: actions/setup-node@v3
+        with:
+          node-version: ${{ matrix.node }}
+          cache: npm
+      - run: npm ci
+      - run: npm run lint
+      - run: |
+          npm run commitlint -- \
+          --verbose \
+          --from `git merge-base origin/master $GITHUB_SHA`
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -7,7 +7,7 @@ on:
 
 jobs:
   linux:
-    name: "Node ${{ matrix.node }} on Linux: Test and Lint"
+    name: "Node ${{ matrix.node }} on Linux: Test"
     runs-on: ubuntu-latest
     strategy:
       fail-fast: false
@@ -27,16 +27,12 @@ jobs:
           node-version: ${{ matrix.node }}
           cache: npm
       - run: npm ci
-      - run: |
-          npm run commitlint -- \
-          --verbose \
-          --from `git merge-base origin/master $GITHUB_SHA`
       - run: npm run build
       - run: npm run test:unit
       - run: npm run test:e2e
       - run: npm run test:integration
   windows:
-    name: "Node ${{ matrix.node }} on Windows: Test and Lint"
+    name: "Node ${{ matrix.node }} on Windows: Test"
     runs-on: windows-latest
     defaults:
       run: