Skip to content

fix: run CI on all pull requests, not just those targeting main#944

Merged
kdaviduik merged 1 commit intomainfrom
03-09-fix_run_ci_on_all_pull_requests_not_just_those_targeting_main
Mar 11, 2026
Merged

fix: run CI on all pull requests, not just those targeting main#944
kdaviduik merged 1 commit intomainfrom
03-09-fix_run_ci_on_all_pull_requests_not_just_those_targeting_main

Conversation

@kdaviduik
Copy link
Copy Markdown
Contributor

The pull_request trigger had branches: [main], which meant CI only
ran on PRs whose base branch is main. Stacked PRs (which target
intermediate branches) never got CI coverage. Remove the branch
filter so CI runs on every PR regardless of base branch.

The push trigger retains branches: [main] — we only want push
CI on main merges, not on every branch push.

The pull_request trigger had `branches: [main]`, which meant CI only
ran on PRs whose base branch is main. Stacked PRs (which target
intermediate branches) never got CI coverage. Remove the branch
filter so CI runs on every PR regardless of base branch.

The push trigger retains `branches: [main]` — we only want push
CI on main merges, not on every branch push.
@kdaviduik kdaviduik force-pushed the 03-09-fix_run_ci_on_all_pull_requests_not_just_those_targeting_main branch from bba962b to 8588844 Compare March 10, 2026 01:37
@kdaviduik kdaviduik force-pushed the kd-update-to-typescript branch from 222f126 to 39ab920 Compare March 10, 2026 01:37
@kdaviduik kdaviduik requested a review from a team March 10, 2026 03:19
Base automatically changed from kd-update-to-typescript to main March 11, 2026 14:24
@kdaviduik kdaviduik merged commit 24caeb9 into main Mar 11, 2026
5 of 7 checks passed
@kdaviduik kdaviduik deleted the 03-09-fix_run_ci_on_all_pull_requests_not_just_those_targeting_main branch March 11, 2026 15:48
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.

3 participants