Skip to content

fix(ci): run test and typecheck on integration-v2 pushes - #415

Closed
shuv1337 wants to merge 1 commit into
integration-v2from
cursor/ci-integration-v2-61f7
Closed

shuv1337 wants to merge 1 commit into
integration-v2from
cursor/ci-integration-v2-61f7

Conversation

@shuv1337

@shuv1337 shuv1337 commented Oct 4, 2026

Copy link
Copy Markdown
Owner

Issue for this PR

Fixes #409

Type of change

  • Bug fix
  • New feature
  • Refactor / code improvement
  • Documentation

What does this PR do?

Pushes to integration-v2 never started test or typecheck, because those workflows only listed v2, which this fork does not have. This adds integration-v2 to both push branch lists and treats refs/heads/integration-v2 like refs/heads/v2 in the test concurrency group, so a newer default-branch push does not cancel the in-progress run.

v2 stays in the lists so the upstream trigger still matches. typecheck has no default-branch concurrency exception today, so that file only gains the push branch.

Both workflows are already active. This does not re-enable them. Turning them back on if they are disabled later is an owner decision.

The e2e job still skips only v2. An integration-v2 push that affects @opencode/app will run Playwright. That skip is unchanged here.

Papercut: pc_409af97187dc

How did you verify your code works?

Parsed both workflow files as YAML. Confirmed with the Actions API that the default branch is integration-v2, that test and typecheck are active, and that recent runs are publish dispatches rather than pushes of those workflows. Did not dispatch Actions.

Screenshots / recordings

N/A. Workflow trigger change only.

Checklist

  • I have tested my changes locally
  • I have not included unrelated changes in this PR
Open in Web Open in Cursor 

Pushes to the fork default branch never matched the v2-only push
triggers, so those workflows did not run. Keep each default-branch
test run instead of cancelling it.

Papercut: pc_409af97187dc

Co-authored-by: shuv <shuv1337@users.noreply.github.com>
@shuv1337

shuv1337 commented Oct 5, 2026

Copy link
Copy Markdown
Owner Author

Closing as superseded: the merged #420 already implements this fix and conflicts with this branch (shuvbro triage, 2026-10-04). Reopen if anything here is still missing.

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.

Papercut: shuv1337/shuvcode pushes to integration-v2 (the fork's default branch) run no CI:…

2 participants