workflows: split out dev checks into jobs

Split out the dev checks into multiple jobs, the benefit of this is that it
will be easier to see which check is failing and why.

Author: kingsleyzissou

.github/workflows/dev-checks.yml

@@ -7,10 +7,14 @@ on:
     branches: [ "main" ]
   merge_group:
 
+concurrency:
+  group: ${{github.workflow}}-${{ github.ref }}
+  cancel-in-progress: true
+
 jobs:
-  dev-check:
+  build:
+    name: Build Check
     runs-on: ubuntu-latest
-
     steps:
     - uses: actions/checkout@v4
     - name: Use Node.js 20
@@ -20,11 +24,60 @@ jobs:
         cache: 'npm'
     - name: Install dependencies
       run: npm ci
-    - name: Check for manual changes to API
-      run: npm run api:generate && [ -z "$(git status --porcelain=v1 2>/dev/null)" ] && echo "✓ No manual API changes." || echo "✗ API manually changed, please refer to the README for the procedure to follow for programmatically generated API endpoints." && [ -z "$(git status --porcelain=v1 2>/dev/null)" ]
-    - name: Check for circular dependencies
-      run: npm run circular
     - name: Run build
       run: npm run build
+
+  lint-checks:
+    name: Linting Checks
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v4
+    - name: Use Node.js 20
+      uses: actions/setup-node@v4
+      with:
+        node-version: 20
+        cache: 'npm'
+    - name: Install dependencies
+      run: npm ci
     - name: Run lint check
       run: npm run lint
+
+  circular-dependencies:
+    name: Circular Dependencies Check
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v4
+    - name: Use Node.js 20
+      uses: actions/setup-node@v4
+      with:
+        node-version: 20
+        cache: 'npm'
+    - name: Install dependencies
+      run: npm ci
+    - name: Check for circular dependencies
+      run: npm run circular
+
+  api-changes:
+    name: Manual API Changes Check
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v4
+    - name: Use Node.js 20
+      uses: actions/setup-node@v4
+      with:
+        node-version: 20
+        cache: 'npm'
+    - name: Install dependencies
+      run: npm ci
+    - name: Check for manual changes to API
+      run: |
+        npm run api:generate
+        if [ -n "$(git status --porcelain)" ]; then
+          echo
+          echo "✓ No manual API changes."
+          exit 0
+        else
+          echo
+          echo "✗ API manually changed, please refer to the README for the procedure to follow for programmatically generated API endpoints."
+          exit 1
+        fi

.github/workflows/playwright.yml

@@ -6,6 +6,12 @@ on:
   workflow_dispatch:
   merge_group:
 
+# this prevents multiple jobs from the same pr
+# running when new changes are pushed.
+concurrency:
+  group: ${{github.workflow}}-${{ github.ref }}
+  cancel-in-progress: true
+
 jobs:
   playwright-tests:
     runs-on:

.github/workflows/unit-tests.yml

@@ -7,6 +7,12 @@ on:
     branches: [ "main" ]
   merge_group:
 
+# this prevents multiple jobs from the same pr
+# running when new changes are pushed.
+concurrency:
+  group: ${{github.workflow}}-${{ github.ref }}
+  cancel-in-progress: true
+
 jobs:
   unit-tests:
     name: Service Unit Tests