[Stack 2/25] CI: run server and E2E tests on PRs targeting jc/** branches#2458
[Stack 2/25] CI: run server and E2E tests on PRs targeting jc/** branches#2458
Conversation
Stacked PRs target jc/* branches, not edge/stable, so server integration tests and Cypress E2E tests were not running on them. Add jc/** to pull_request.branches so stack PRs get full CI coverage (python-ci.yml already has this from a prior change). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
jucor
changed the title
There was a problem hiding this comment.
Pull request overview
Updates CI workflow triggers so server integration (Jest) and Cypress E2E workflows also run for pull requests whose base branch matches jc/**, aligning with the stacked-PR workflow described in the PR.
Changes:
- Expand
pull_request.branchesfilters in Jest server integration workflow to includejc/** - Expand
pull_request.branchesfilters in Cypress E2E workflow to includejc/**
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| .github/workflows/jest-server-test.yml | Adds jc/** to PR base-branch filters so server integration tests run on stacked PRs |
| .github/workflows/cypress-tests.yml | Adds jc/** to PR base-branch filters so Cypress E2E tests run on stacked PRs |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@ballPointPenguin This one FYI, which I'll merge now: it just makes sure all the PRs in m stack are getting CI-ed properly. |
Summary
jc/**topull_request.branchesinjest-server-test.ymlandcypress-tests.ymljc/*branches, notedge/stable, so server integration tests and Cypress E2E tests were not running on thempython-ci.ymlalready has this from a prior change; this brings the other two heavy CI workflows in lineTest plan
Generated with Claude Code