Skip to content

Switch CI backend dependency installation from pip to uv #561

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
gkorland merged 7 commits into from
Mar 10, 2026
Merged

Switch CI backend dependency installation from pip to uv #561

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
9f56896
Initial plan
Copilot Mar 10, 2026
8610bb9
ci: switch workflow backend installs to uv sync
Copilot Mar 10, 2026
7aaa545
ci: preserve nextjs test extras with uv sync
Copilot Mar 10, 2026
c1b27a3
ci: fix playwright python import path
Copilot Mar 10, 2026
25f5344
Merge branch 'staging' into copilot/switch-ci-to-uv-sync
gkorland Mar 10, 2026
b9ccd3e
Merge branch 'staging' into copilot/switch-ci-to-uv-sync
gkorland Mar 10, 2026
3c44cd8
Merge branch 'staging' into copilot/switch-ci-to-uv-sync
gkorland Mar 10, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 6 additions & 1 deletion .github/workflows/nextjs.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -46,8 +46,13 @@ jobs:
with:
python-version: "3.12"

- name: Install uv
uses: astral-sh/setup-uv@v5
with:
version: "latest"

Copilot AI Mar 10, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Consider pinning uv to a specific version instead of "latest" so CI runs remain reproducible and uv upgrades are intentional.

Suggested change
version: "latest"
version: "0.4.28"

Copilot uses AI. Check for mistakes.

- name: Install backend dependencies
run: pip install -e ".[test]"
run: uv sync --extra test

docker-build:
runs-on: ubuntu-latest
Expand Down
13 changes: 10 additions & 3 deletions .github/workflows/playwright.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,8 @@ jobs:
test:
timeout-minutes: 60
runs-on: ubuntu-latest
env:
PYTHONPATH: ${{ github.workspace }}
strategy:
fail-fast: false
matrix:
Expand All @@ -37,8 +39,13 @@ jobs:
with:
python-version: '3.12'

- name: Install uv
uses: astral-sh/setup-uv@v5
with:
version: "latest"

Copilot AI Mar 10, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Using version: "latest" for uv makes CI behavior non-deterministic across time. Consider pinning uv to a specific version so workflow runs remain reproducible (and upgrades are explicit).

Suggested change
version: "latest"
version: "0.5.1"

Copilot uses AI. Check for mistakes.

- name: Install backend dependencies
run: pip install -e .
run: uv sync

- name: Install frontend dependencies
working-directory: ./app
Expand All @@ -51,7 +58,7 @@ jobs:
run: npm run build

- name: Seed test data into FalkorDB
run: python e2e/seed_test_data.py
run: uv run python e2e/seed_test_data.py

- name: Install Playwright npm dependencies
run: |
Expand Down Expand Up @@ -81,7 +88,7 @@ jobs:
CODE_GRAPH_PUBLIC: "1"
MODEL_NAME: "openai/gpt-4.1-mini"
run: |
flask --app api/index.py run --host 0.0.0.0 --port 5000 &
uv run flask --app api/index.py run --host 0.0.0.0 --port 5000 &
# Wait for Flask to be ready
timeout 30 bash -c 'until curl -s http://localhost:5000/ > /dev/null 2>&1; do sleep 0.5; done'
npx playwright test --shard=${{ matrix.shard }}/2 --reporter=dot,list
Expand Down
Loading