ci: use continuous target-branch check for master - #1330
Conversation
This repo releases from master, but the gitflow variant of the shared check only accepts main/ and build/ branches, so it failed on every pull request. Switch to the continuous (trunk-based) variant, which accepts master alongside main, build/ and chore/dependabot.
PR SummaryLow Risk Overview The reusable workflow job is renamed to The check’s displayed name may change (e.g. wording around main vs master); branch protection pinned to the old check name would need updating if it were ever required—which it effectively wasn’t while the check always failed on Reviewed by Cursor Bugbot for commit 321e42d. Bugbot is set up for automated code reviews on this repo. Configure here. |
|




Background
The
Check PR for semantic target branchcheck fails on every pull request intomaster, including ones that were merged (#1274, #1254, and currently #1329).The repo calls the gitflow variant of the shared check:
That workflow only accepts
main,build/andchore/dependabotbranches, and takes no inputs, somastercan never satisfy it:This repo's default branch is
masterand it releases frommaster, so the gitflow variant is simply the wrong one.What Has Changed
Switch to the continuous (trunk-based) variant that mParticle already publishes, which accepts
masterin addition tomain,build/andchore/dependabot:No other job is touched. Note the displayed check name changes from
Confirm that target branch for PR is main or build/toConfirm that target branch for PR is main or master, so any branch-protection rule pinned to the old name would need updating — though since the old check has never passed onmaster, it cannot currently be a required check.Test plan
master, so the check on this PR is itself the test: it should now passScreenshots/Video