Skip to content

Commit 4ce4b87

Browse files
authored
Drop codegen-* heads from test-pull-request branches filter (#132)
Addresses Copilot review finding on PR #131: `branches:` under `pull_request` matches the PR's *base* branch, and `codegen-main`/`codegen-develop` are head-only branches by design (never a PR base), so listing them was redundant. Restricts the list to the protected base branches.
1 parent a123281 commit 4ce4b87

1 file changed

Lines changed: 5 additions & 1 deletion

File tree

.github/workflows/test-pull-request.yml

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,11 @@ name: Test pull request action
22

33
on:
44
pull_request:
5-
branches: [ main, develop, codegen-main, codegen-develop ]
5+
# The `branches:` filter under `pull_request` matches the PR's BASE
6+
# branch. `codegen-main` and `codegen-develop` are head-only branches
7+
# (the codegen workflow opens PRs *from* them into `main` / `develop`)
8+
# so they're never bases — only the protected base branches go here.
9+
branches: [ main, develop ]
610
workflow_dispatch:
711

812
concurrency:

0 commit comments

Comments
 (0)