Align GitHub Actions backend dependency sync with Makefile#604
Align GitHub Actions backend dependency sync with Makefile#604
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
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 touv sync --all-extras..github/workflows/build.yml: switch backend install step fromuv sync --extra testtouv 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.
Co-authored-by: gkorland <753206+gkorland@users.noreply.github.com>
18682aa to
d032c6f
Compare
|
@CodeRabbit review |
✅ Actions performedReview triggered.
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughCI workflows updated to use Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
CI and local development were using different
uv syncscopes 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 asmake install.What changed
.github/workflows/build.yml.github/workflows/playwright.ymluv sync --all-extras, matching the Makefile.Why this matters
make installWorkflow alignment
build.yml:uv sync --extra testplaywright.yml:uv syncuv sync --all-extrasOriginal prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.
Summary by CodeRabbit