microsoft / microsoft/hve-core

feat: verify branch protection requires only the pr-validation-success gate

Open
#2,706 0 comments 1 reaction 1 assignee View on GitHub

@chalew101 is already working on this.

Since Aug 13, 2026.

Dominant language
Python
Stars
1.5k
Forks
301
Avg merge
3d 3h
Merged PRs (30d)
92

Description

Issue Description

Confirm with an org/repo admin that GitHub branch protection on main (and release/prerelease, release/stable) lists only pr-validation-success as a required status check, not individual job names from pr-validation.yml.

This matters because the entire "every job must appear in pr-validation-success's needs:" model (enforced by scripts/security/Test-PrValidationGate.ps1) only works if pr-validation-success is the actual merge gate. If branch protection instead requires individual job names directly, then:

  • A new job added to the workflow but not required by branch protection could fail without blocking merge, even if it's correctly listed in needs:
  • The aggregator gate becomes redundant/bypassable
Additional Context

Blocked on: requires repo admin access to read/verify Settings > Branches > Branch protection rules (the gh api repos/.../branches/main/protection call returned 403 for a non-admin token during discovery).

Deliverable: a short confirmation (or correction) documented back on this issue, and if misconfigured, a follow-up to align branch protection with the single-gate model.

Parent: #2703

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.