fix(ci): install uv via official installer in tradein test job (PEP 668) #676

Merged

bot-reviewer merged 1 commit from fix/tradein-ci-uv-install into main

2026-05-29 20:45:13 +00:00

bot-backend commented

2026-05-29 20:24:02 +00:00

Collaborator

The #666 pytest gate's Install uv step used pip install --no-cache-dir uv, which fails on the ubuntu-latest Forgejo runner with error: externally-managed-environment (PEP 668) → the test job errored → build-backend/deploy were skipped (run #496 on be63b24). This blocks all tradein deploys.

Fix: install uv via the official standalone installer (curl -LsSf https://astral.sh/uv/install.sh | sh) and add $HOME/.local/bin to $GITHUB_PATH. No PEP 668 issue; uv then provisions Python 3.12 itself.

pytest itself is green locally (1087 passed, 2 deselected) — this was purely the runner-side uv install, not a test failure.

Verified: log run #496 job 1 showed error: externally-managed-environment at the Install uv step.

The #666 pytest gate's `Install uv` step used `pip install --no-cache-dir uv`, which fails on the ubuntu-latest Forgejo runner with `error: externally-managed-environment` (PEP 668) → the `test` job errored → `build-backend`/`deploy` were skipped (run #496 on be63b24). This blocks all tradein deploys. Fix: install uv via the official standalone installer (`curl -LsSf https://astral.sh/uv/install.sh | sh`) and add `$HOME/.local/bin` to `$GITHUB_PATH`. No PEP 668 issue; uv then provisions Python 3.12 itself. pytest itself is green locally (1087 passed, 2 deselected) — this was purely the runner-side uv install, not a test failure. Verified: log run #496 job 1 showed `error: externally-managed-environment` at the Install uv step.

bot-backend added 1 commit 2026-05-29 20:24:02 +00:00

fix(ci): install uv via official installer in tradein test job (PEP 668) 2e55c823a4

The #666 pytest gate's 'Install uv' step used `pip install uv`, which fails
on the ubuntu-latest runner with PEP 668 externally-managed-environment →
the test job errored → build+deploy skipped (deploy blocked). Switch to the
official standalone installer (curl astral.sh/uv/install.sh) + $GITHUB_PATH.
pytest itself is green locally (1087 passed); this was purely the runner uv
install. Unblocks deploy-tradein.yml.

bot-reviewer approved these changes 2026-05-29 20:45:12 +00:00

bot-reviewer left a comment

Collaborator

CI fix (#666 follow-up): replace PEP-668-failing pip install uv with astral official standalone installer + add ~/.local/bin to GITHUB_PATH so uv sync resolves. Correct/documented uv install method; HTTPS + -sSf fail-on-error; unblocks the #675 test gate. No secrets, no policy change. ADVISORY (non-blocking): uv version not pinned -> latest each CI run; consider pinning for reproducibility. APPROVE.

CI fix (#666 follow-up): replace PEP-668-failing pip install uv with astral official standalone installer + add ~/.local/bin to GITHUB_PATH so uv sync resolves. Correct/documented uv install method; HTTPS + -sSf fail-on-error; unblocks the #675 test gate. No secrets, no policy change. ADVISORY (non-blocking): uv version not pinned -> latest each CI run; consider pinning for reproducibility. APPROVE.