Skip to content

ci: run PR checks on stacked PRs (not just PRs targeting main)#1063

Merged
svarlamov merged 3 commits intomainfrom
fix-stacked-pr-ci
Apr 11, 2026
Merged

ci: run PR checks on stacked PRs (not just PRs targeting main)#1063
svarlamov merged 3 commits intomainfrom
fix-stacked-pr-ci

Conversation

@svarlamov
Copy link
Copy Markdown
Member

@svarlamov svarlamov commented Apr 11, 2026

Summary

  • Remove branches: [main] from the pull_request trigger in test.yml, lint-format.yml, and coverage.yml
  • CI (tests, lint/format, coverage) now runs on all pull requests regardless of their target branch
  • Stacked PRs (targeting a feature branch instead of main) will now get the same CI feedback as regular PRs

The push and merge_group branch filters are left unchanged — those still only run on main.

Test plan

  • Open a stacked PR (one targeting a non-main branch) and verify that the Test, Lint & Format, and Coverage workflows are triggered
  • Verify that PRs targeting main still run the same checks as before

🤖 Generated with Claude Code


Open with Devin

svarlamov and others added 3 commits April 11, 2026 18:59
Covers 9 new IDE/editor integrations (Windsurf, Zed, Sublime Text,
Eclipse, Xcode, Vim, Neovim, Visual Studio, Notepad++) to be
implemented as parallel worktree PRs by subagents.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Remove `branches: [main]` filter from the `pull_request` trigger in
test.yml, lint-format.yml, and coverage.yml so that CI runs on all
pull requests regardless of their target branch. This ensures stacked
PRs get the same test/lint/coverage feedback as PRs targeting main.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@svarlamov svarlamov merged commit 8aabfb5 into main Apr 11, 2026
21 of 23 checks passed
@svarlamov svarlamov deleted the fix-stacked-pr-ci branch April 11, 2026 23:11
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 1 additional finding.

Open in Devin Review

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.

1 participant