Skip to content

fix(windows-wsus): declare the repo's explicit CODEOWNERS surfaces - #59

Open
NWarila wants to merge 1 commit into
mainfrom
fix/windows-wsus-codeowners
Open

fix(windows-wsus): declare the repo's explicit CODEOWNERS surfaces#59
NWarila wants to merge 1 commit into
mainfrom
fix/windows-wsus-codeowners

Conversation

@NWarila

@NWarila NWarila commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

The org default repo_default_codeowners = "* @NWarila" overwrote windows-wsus's CODEOWNERS on sync (chore(codeowners): sync CODEOWNERS via terraform, windows-wsus 5e9a4ad). That repository's required gate — scripts/check-workflow-trigger.sh — asserts the exact seven-rule ownership of its automation/security surfaces, so its protected main is now failing its own required check on every PR.

Per-repo codeowners: is the documented override for repo_default_codeowners (precedence 1, used verbatim). This declares the previous seven-rule file byte-for-byte so the next sync restores the repository to self-consistency.

Worth a follow-up sweep: any sibling repo that pins its CODEOWNERS content in a gate (the pattern was cloned from secure-wazuh) has the same exposure.

The org-wide repo_default_codeowners fallback flattened windows-wsus's CODEOWNERS to the single general rule, but that repository's required gate asserts exact ownership of its automation and security surfaces, so protected main went permanently red the moment the sync landed. Per-repo codeowners is the documented override for exactly this case; the block restores the previous seven-rule file verbatim, and the next sync makes the repository self-consistent again.
@NWarila
NWarila enabled auto-merge (squash) August 10, 2026 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant