Skip to content

[orbit-agent] Swap pip → uv in CI workflow(s)#22

Merged
haasonsaas merged 1 commit intomainfrom
ci/pip-to-uv
Apr 25, 2026
Merged

[orbit-agent] Swap pip → uv in CI workflow(s)#22
haasonsaas merged 1 commit intomainfrom
ci/pip-to-uv

Conversation

@haasonsaas
Copy link
Copy Markdown
Contributor

Summary

  • Replace python -m pip install -U pip + pip install -r requirements-dev.lock with uv pip install --system -r requirements-dev.lock in .github/workflows/ci.yml.
  • Add astral-sh/setup-uv@08807647e7069bb48b6ef5acd8ec9567f424441b (v8.1.0, SHA-pinned) after Set up Python; applies to both matrix legs (3.11, 3.12).
  • scripts/bootstrap_venv.sh is intentionally left alone — Wave 3, out of scope.

Cross-ref: evalops/maestro-internal#1492

Test plan

  • CI matrix run on this PR is green for python-version 3.11 and 3.12.
  • black --check, ruff check, and pytest -q all pass against uv-installed deps.

🤖 Generated with Claude Code

Replace `python -m pip install -U pip` + `pip install -r requirements-dev.lock`
with `uv pip install --system -r requirements-dev.lock` in
`.github/workflows/ci.yml`. Adds astral-sh/setup-uv@v8.1.0 (SHA-pinned)
after Set up Python on both matrix legs (3.11, 3.12). Drops the bare-pip
self-upgrade per the migration recipe.

`scripts/bootstrap_venv.sh` is intentionally left alone (Wave 3, out of scope).

Cross-ref: evalops/maestro-internal#1492

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@cursor
Copy link
Copy Markdown

cursor Bot commented Apr 25, 2026

PR Summary

Low Risk
Low risk: only changes the CI dependency installation mechanism; failures would surface as CI breakage rather than runtime behavior changes.

Overview
CI now installs Python dependencies with uv instead of pip by adding a SHA-pinned astral-sh/setup-uv step and switching the install command to uv pip install --system -r requirements-dev.lock for both Python 3.11/3.12 matrix runs.

Reviewed by Cursor Bugbot for commit d34d56d. Bugbot is set up for automated code reviews on this repo. Configure here.

@haasonsaas haasonsaas merged commit ee0343e into main Apr 25, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant