Problem
PR #58 was merged to main as commit 30de716 using the admin bypass (enforce_admins: true blocked the normal gh pr merge). Branch protection was lowered to allow the merge.
The required status checks (ceo-audit + tsc-check) were DISABLED during the merge, so we never verified they pass on the new commits.
Goal
Re-enable branch protection with the required status checks and verify they pass on main HEAD 30de716.
Acceptance Criteria
Suggested steps
# 1. Re-enable protection
gh api --method PUT repos/OpenSIN-Code/SIN-Code-WebUI-v2/branches/main/protection \
--input <(echo '{
"restrictions":null,
"required_status_checks":{"strict":true,"contexts":["ceo-audit","tsc-check"]},
"enforce_admins":true,
"required_pull_request_reviews":{"required_approving_review_count":1}
}')
# 2. Trigger a fresh CI run (touch a file or push empty commit)
git commit --allow-empty -m "ci: re-trigger checks on main"
git push origin main --force # with admin bypass
# 3. Watch the checks
gh run watch
# 4. Verify all green
gh pr checks 2>/dev/null || gh api repos/OpenSIN-Code/SIN-Code-WebUI-v2/commits/main/status
Related
Problem
PR #58 was merged to main as commit
30de716using the admin bypass (enforce_admins: trueblocked the normalgh pr merge). Branch protection was lowered to allow the merge.The required status checks (ceo-audit + tsc-check) were DISABLED during the merge, so we never verified they pass on the new commits.
Goal
Re-enable branch protection with the required status checks and verify they pass on
mainHEAD30de716.Acceptance Criteria
ceo-audit+tsc-checkas required contexts30de716gh pr mergeworks for the next PR (no admin bypass needed)Suggested steps
Related
.github/workflows/ceo-audit.yml,.github/workflows/tsc-check.yml