Skip to content

Commit 1fc68f4

Browse files
fix(ci): integration workflow installs requirements.txt, not 'pip install -e .'
The repo is not a pip package (no setup.py/pyproject.toml; it runs via PYTHONPATH, as ci.yaml documents), so integration.yaml's 'pip install -e .' always errored ('does not appear to be a Python project') whenever the Mac M4 integration job actually ran. #157 triggered that job (it adds an integration test), exposing the pre-existing break. Install runtime deps from requirements.txt instead; the suite already runs with PYTHONPATH=.:sdks/python. Co-authored-by: FluffyAIcode <FluffyAIcode@users.noreply.github.com>
1 parent 15741c1 commit 1fc68f4

1 file changed

Lines changed: 5 additions & 1 deletion

File tree

.github/workflows/integration.yaml

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,11 @@ jobs:
8585
# The runner is expected to have a long-lived venv.
8686
# If a per-run venv is preferred, swap to ``python3 -m venv .venv``.
8787
python3 -m pip install --upgrade pip
88-
python3 -m pip install -e .
88+
# The repo runs via PYTHONPATH (see ci.yaml) — it is NOT a pip package
89+
# (no setup.py/pyproject.toml), so install runtime deps from
90+
# requirements.txt rather than an editable `-e .` (which errors with
91+
# "does not appear to be a Python project").
92+
python3 -m pip install -r requirements.txt
8993
python3 -m pip install pytest pytest-asyncio pytest-timeout coverage
9094
9195
- name: Run integration suite

0 commit comments

Comments
 (0)