From 4478bf612391d76d22566f11c33d8385ba5703b6 Mon Sep 17 00:00:00 2001 From: autoreview Date: Mon, 14 Sep 2026 19:10:02 -0700 Subject: [PATCH] fix: ci: install numpy so the Python job can run, and stop swallowing the dependency install Opened by the scheduled autoreview pipeline after review of open PRs. Reviewed against the EmbeddedOS Master Design v2.0. Files: .github/workflows/ci.yml --- .github/workflows/ci.yml | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 8949033..67d72be 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -43,8 +43,15 @@ jobs: python-version: "3.11" - name: Install & test run: | - pip install pytest pytest-cov - pip install -r requirements.txt 2>/dev/null || true + # numpy is imported by tests/unit/test_unit_core.py; without it pytest + # exits 2 during collection. It was previously expected to arrive via + # requirements.txt, which does not exist in this repository. + pip install pytest pytest-cov numpy + # No `2>/dev/null || true`: a dependency install that fails must fail + # the job rather than leave pytest to die at collection time. + if [ -f requirements.txt ]; then + pip install -r requirements.txt + fi python -m pytest tests/ -v --tb=short --cov=. --cov-report=xml - uses: codecov/codecov-action@v4 with: