refactor: Separate linting from unit tests workflows

- Added dedicated lint.yml workflow for ruff and flake8 linting
  - Single Python 3.11 version for consistency
  - Uses cached pip dependencies for speed
  - Runs ruff check and flake8 (syntax + style)
- Updated tests.yml to focus only on unit tests
  - Removed linting steps
  - Added cached pip dependencies
  - Same Python matrix: 3.8, 3.10, 3.12
- Both workflows trigger on push/PR to main/dev + manual
- Workflows now run in parallel for faster feedback

Author: minaskar

.github/workflows/lint.yml

@@ -0,0 +1,49 @@
+# Lint code with ruff and flake8
+name: Lint Code
+
+on:
+  push:
+    branches: ["main", "dev"]
+  pull_request:
+    branches: ["main", "dev"]
+  workflow_dispatch:
+
+jobs:
+  lint:
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout source
+        uses: actions/checkout@v4
+
+      - name: Set up Python 3.11
+        uses: actions/setup-python@v4
+        with:
+          python-version: "3.11"
+
+      - name: Cache pip dependencies
+        uses: actions/cache@v3
+        with:
+          path: ~/.cache/pip
+          key: ${{ runner.os }}-pip-lint-${{ hashFiles('requirements.txt') }}
+          restore-keys: |
+            ${{ runner.os }}-pip-lint-
+
+      - name: Install linters
+        run: |
+          python -m pip install --upgrade pip
+          python -m pip install ruff flake8
+
+      - name: Lint with ruff
+        run: |
+          ruff check --output-format=github .
+
+      - name: Lint with flake8 (syntax)
+        run: |
+          # stop the build if there are Python syntax errors or undefined names
+          flake8 . --count --select=E9,F63,F7,F82 --show-source --statistics
+
+      - name: Lint with flake8 (style)
+        run: |
+          # exit-zero treats all errors as warnings
+          flake8 . --count --exit-zero --max-complexity=10 --max-line-length=127 --statistics

.github/workflows/tests.yml

@@ -1,4 +1,4 @@
-# This workflow will install Python dependencies, run tests, and lint with a variety of Python versions
+# This workflow runs unit tests across multiple Python versions
 name: Run Tests
 
 on:
@@ -17,30 +17,27 @@ jobs:
         python-version: ["3.8", "3.10", "3.12"]
 
     steps:
-      - uses: actions/checkout@v4
+      - name: Checkout source
+        uses: actions/checkout@v4
 
       - name: Set up Python ${{ matrix.python-version }}
         uses: actions/setup-python@v4
         with:
           python-version: ${{ matrix.python-version }}
 
+      - name: Cache pip dependencies
+        uses: actions/cache@v3
+        with:
+          path: ~/.cache/pip
+          key: ${{ runner.os }}-pip-tests-${{ matrix.python-version }}-${{ hashFiles('requirements.txt') }}
+          restore-keys: |
+            ${{ runner.os }}-pip-tests-${{ matrix.python-version }}-
+
       - name: Install dependencies
         run: |
           python -m pip install --upgrade pip
-          python -m pip install flake8 ruff
           if [ -f requirements.txt ]; then pip install -r requirements.txt; fi
 
-      - name: Lint with ruff
-        run: |
-          ruff check --output-format=github .
-
-      - name: Lint with flake8
-        run: |
-          # stop the build if there are Python syntax errors or undefined names
-          flake8 . --count --select=E9,F63,F7,F82 --show-source --statistics
-          # exit-zero treats all errors as warnings. The GitHub editor is 127 chars wide
-          flake8 . --count --exit-zero --max-complexity=10 --max-line-length=127 --statistics
-
       - name: Run tests
         run: |
           python -m unittest discover tests