Skip to content

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

Open
kdaviduik wants to merge 1 commit intokd-update-to-typescriptfrom
03-09-fix_run_ci_on_all_pull_requests_not_just_those_targeting_main
Open

fix: run CI on all pull requests, not just those targeting main#944
kdaviduik wants to merge 1 commit intokd-update-to-typescriptfrom
03-09-fix_run_ci_on_all_pull_requests_not_just_those_targeting_main

Conversation

@kdaviduik
Copy link
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.

Copy link
Contributor Author

kdaviduik commented Mar 9, 2026

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
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