Skip to content

Align GitHub Actions backend dependency sync with Makefile#604

Open
Copilot wants to merge 3 commits intostagingfrom
copilot/align-ci-and-makefile-tools
Open

Align GitHub Actions backend dependency sync with Makefile#604
Copilot wants to merge 3 commits intostagingfrom
copilot/align-ci-and-makefile-tools

Conversation

Copy link
Contributor

Copilot AI commented Mar 10, 2026

CI and local development were using different uv sync scopes for Python dependencies, which can produce different lockfile installs between workflows and the Makefile. This updates the affected GitHub Actions workflows to use the same backend dependency set as make install.

  • What changed

    • Updated the backend dependency install step in:
      • .github/workflows/build.yml
      • .github/workflows/playwright.yml
    • Both workflows now use uv sync --all-extras, matching the Makefile.
  • Why this matters

    • Removes drift between CI and local setup
    • Ensures workflow runs resolve the same optional dependency set developers get from make install
  • Workflow alignment

    • Before:
      • build.yml: uv sync --extra test
      • playwright.yml: uv sync
    • After:
      • both workflows: uv sync --all-extras
- name: Install backend dependencies
  run: uv sync --all-extras
Original prompt

This section details on the original issue you should resolve

<issue_title>CI uses pip install while Makefile uses uv sync</issue_title>
<issue_description>## Description

There is an inconsistency between CI and local development tooling:

CI workflows use pip:

  • .github/workflows/nextjs.yml: pip install -e ".[test]"
  • .github/workflows/playwright.yml: pip install -e .

Makefile uses uv:

  • make install: uv sync --all-extras
  • make test: uv run python -m pytest
  • make run-dev: uv run flask ...

This can lead to different dependency resolutions and hard-to-reproduce CI failures.

Suggested Fix

Align CI workflows with the Makefile by using uv in CI:

- run: pip install uv
- run: uv sync --all-extras

Context

Found during code review of PR #522.</issue_description>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

@vercel
Copy link

vercel bot commented Mar 10, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
code-graph Error Error Mar 10, 2026 9:28pm

Request Review

Co-authored-by: gkorland <753206+gkorland@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix inconsistency between CI and Makefile dependency installation Align GitHub Actions backend dependency sync with Makefile Mar 10, 2026
@gkorland gkorland marked this pull request as ready for review March 18, 2026 07:49
@gkorland gkorland requested a review from Copilot March 18, 2026 07:49
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates GitHub Actions workflows to install the same Python dependency set in CI as developers get locally via make install, reducing dependency-resolution drift across environments.

Changes:

  • .github/workflows/playwright.yml: switch backend install step to uv sync --all-extras.
  • .github/workflows/build.yml: switch backend install step from uv sync --extra test to uv sync --all-extras.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
.github/workflows/playwright.yml Aligns Playwright CI backend dependency sync scope with make install by using --all-extras.
.github/workflows/build.yml Aligns build CI backend dependency sync scope with make install by using --all-extras.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

CI uses pip install while Makefile uses uv sync

3 participants