Skip to content

Fix empty-sample fork trusted gate - #968

Merged
Brandon Miller (brandom-msft) merged 1 commit into
mainfrom
brandom-msft-fix-empty-sample-fork-gate
Sep 9, 2026
Merged

Brandon Miller (brandom-msft) merged 1 commit into
mainfrom
brandom-msft-fix-empty-sample-fork-gate

Conversation

@brandom-msft

Copy link
Copy Markdown
Member

Summary

  • allow a valid empty detector result (has_changes=false, count=0, samples=[]) to satisfy the single required validate / trusted context for fork and same-repository pull requests
  • retain explicit maintainer-promotion failure for affected-sample forks
  • preserve same-repository build readiness/live-service behavior and fail-closed detector-contract handling
  • extend the existing structural regression harness to pin the corrected conditions

Validation

  • bash .github/scripts/test/run-workflow-structure-tests.sh (41 checks passed)

Tracking

ADO 5555657: https://msdata.visualstudio.com/Vienna/_workitems/edit/5555657

This is intentionally the narrow correction surfaced by #932; checkout-free classification and broader validation architecture remain deferred.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@brandom-msft
Brandon Miller (brandom-msft) merged commit ec446db into main Sep 9, 2026
16 checks passed
@brandom-msft
Brandon Miller (brandom-msft) deleted the brandom-msft-fix-empty-sample-fork-gate branch September 9, 2026 23:16
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