Update ci.yml

Author: Diegotistical

.github/workflows/ci.yml

@@ -1,8 +1,3 @@
-# GitHub Actions CI workflow: lint, type-check, test, notebooks, upload artifacts
-# - Python matrix: 3.10, 3.11, 3.12
-# - Uses pip cache to speed up installs
-# - Runs isort (check), Black (check), Flake8 (with bugbear), Mypy, Pytest (with coverage)
-# - Executes notebooks (if any) using jupyter nbconvert
 name: CI
 
 on:
@@ -19,9 +14,6 @@ on:
     - cron: '0 3 * * *'
   workflow_dispatch:
 
-env:
-  PIP_CACHE_DIR: ~/.cache/pip
-
 jobs:
   lint-type-test:
     name: Lint / Type-check / Test (Python ${{ matrix.python-version }})
@@ -30,8 +22,7 @@ jobs:
       matrix:
         python-version: ['3.10', '3.11', '3.12']
     steps:
-      - name: Checkout repo
-        uses: actions/checkout@v4
+      - uses: actions/checkout@v4
 
       - name: Setup Python
         uses: actions/setup-python@v4
@@ -50,96 +41,74 @@ jobs:
         run: |
           python -m pip install --upgrade pip
           if [ -f requirements.txt ]; then pip install -r requirements.txt; fi
-          pip install "black==24.*" "isort==5.*" "flake8==7.*" "flake8-bugbear==23.*" "mypy==1.*" "pytest==8.*" "pytest-cov" "nbval" "jupyter" "coverage"
+          pip install black==24.* isort==5.* flake8==7.* flake8-bugbear==23.* mypy==1.* pytest==8.* pytest-cov nbval jupyter coverage nbconvert nbformat
 
-      - name: Run isort (check)
-        run: |
-          isort --version
-          isort --check-only .
+      - name: Run isort
+        run: isort --check-only .
 
-      - name: Run Black (check)
-        run: |
-          black --version
-          black --check .
+      - name: Run Black
+        run: black --check .
 
-      - name: Run Flake8 (including bugbear checks)
-        run: |
-          flake8 .
+      - name: Run Flake8
+        run: flake8 .
 
       - name: Run Mypy
-        run: |
-          mypy .
+        run: mypy .
 
-      - name: Run Pytest (with coverage)
-        continue-on-error: false
-        run: |
-          pytest --maxfail=1 --disable-warnings -q --cov=. --cov-report=xml
+      - name: Run Pytest with coverage
+        run: pytest --maxfail=1 --disable-warnings -q --cov=. --cov-report=xml
 
-      - name: Upload test/coverage artifacts
-        if: always()
+      - name: Upload test and coverage artifacts
         uses: actions/upload-artifact@v4
         with:
           name: coverage-and-tests-${{ matrix.python-version }}
           path: |
             coverage.xml
             .pytest_cache || true
 
-      - name: Upload coverage to Codecov (optional)
-        if: ${{ secrets.CODECOV_TOKEN != '' }}
-        uses: codecov/codecov-action@v4
-        with:
-          files: coverage.xml
-          token: ${{ secrets.CODECOV_TOKEN }}
-
   notebooks:
-    name: Execute Jupyter notebooks (single runner)
+    name: Execute Jupyter notebooks
     runs-on: ubuntu-latest
     needs: lint-type-test
     strategy:
       matrix:
         python-version: ['3.11']
     steps:
-      - name: Checkout repo
-        uses: actions/checkout@v4
+      - uses: actions/checkout@v4
 
       - name: Setup Python
         uses: actions/setup-python@v4
         with:
           python-version: ${{ matrix.python-version }}
 
-      - name: Cache pip for notebooks
+      - name: Cache pip
         uses: actions/cache@v4
         with:
           path: ~/.cache/pip
           key: notebooks-pip-${{ matrix.python-version }}-${{ hashFiles('**/requirements.txt') }}
           restore-keys: |
             notebooks-pip-${{ matrix.python-version }}-
 
-      - name: Install jupyter & nbconvert
+      - name: Install Jupyter & nbconvert
         run: |
           python -m pip install --upgrade pip
-          if [ -f requirements.txt ]; then pip install -r requirements.txt; fi
-          pip install "jupyter" "nbconvert" "nbformat"
+          pip install jupyter nbconvert nbformat
 
-      - name: Find and execute notebooks
+      - name: Execute notebooks
         run: |
           set -euo pipefail
           mapfile -t NOTEBOOKS < <(git ls-files '*.ipynb' || true)
           if [ ${#NOTEBOOKS[@]} -eq 0 ]; then
-            echo "No notebooks found in repository — skipping execution."
+            echo "No notebooks found — skipping."
             exit 0
           fi
-          echo "Found notebooks:"
-          printf '%s\n' "${NOTEBOOKS[@]}"
           for nb in "${NOTEBOOKS[@]}"; do
             echo "Executing $nb"
             out="/tmp/$(basename "$nb" .ipynb)-executed.ipynb"
             jupyter nbconvert --to notebook --execute "$nb" --ExecutePreprocessor.timeout=600 --output "$out"
-            echo "$out"
           done
 
-      - name: Upload executed notebooks (artifact)
-        if: always()
+      - name: Upload executed notebooks
         uses: actions/upload-artifact@v4
         with:
           name: executed-notebooks