Skip to content

ci: rename branch-protection caller job id to branch-protection - #114

Open
rubenvdlinde wants to merge 1 commit into
mainfrom
hotfix/ci-branch-protection-job-id
Open

ci: rename branch-protection caller job id to branch-protection#114
rubenvdlinde wants to merge 1 commit into
mainfrom
hotfix/ci-branch-protection-job-id

Conversation

@rubenvdlinde

Copy link
Copy Markdown
Contributor

GitHub names a reusable-workflow status context <caller-job-id> / <called-job-name>.

This repo's caller job id was not branch-protection, so the shared branch-protection
workflow reported under the wrong context name. The org rulesets (Main Branch Protection
id 14128365, Beta Branch Protection id 14128357) require exactly:

branch-protection / check-branch
quality / Quality Report

No name match means the required context never reported at all — not as a failure, as
nothing. The PR sits at BLOCKED forever, and the absence of the check is visually identical to
the check merely not having finished yet. That is the whole defect: a check's absence looks
exactly like its success is still pending.

This renames the caller job id so the context matches. It does not change what is required and
does not touch any ruleset or branch protection — that is a governance decision, not a code
one.

Irony worth recording: this PR itself has to be admin-merged, because the defect it fixes is
precisely what makes the required context unreportable on the current tip.

…ion`

GitHub names a reusable-workflow status context `<caller-job-id> / <called-job-name>`, so this
caller reported as `check / check-branch` while the org ruleset requires
`branch-protection / check-branch`. The required context never reported at all, leaving PRs to
main/beta permanently BLOCKED.
@github-actions

github-actions Bot commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Quality Report — ConductionNL/openbuild @ 950dcb0

Check PHP Vue Security License Tests
lint
phpcs
phpmd
psalm
phpstan
phpmetrics
eslint
stylelint
build
composer ✅ 100/100
npm ✅ 389/389
PHPUnit ⏭️
Newman ⏭️
Playwright ⏭️

Quality workflow — 2026-08-04 06:17 UTC

Download the full PDF report from the workflow artifacts.

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