Skip to content

ci(security): enforce codeowners review, stale review dismissal, and required status checks on main (#189) - #195

Merged
OdevMatheus merged 1 commit into
mainfrom
ci/enforce-codeowners-and-status-checks
Nov 9, 2026
Merged

OdevMatheus merged 1 commit into
mainfrom
ci/enforce-codeowners-and-status-checks

Conversation

@OdevMatheus

Copy link
Copy Markdown
Owner

Summary

  • Adds global wildcard * @OdevMatheus ownership at the top of .github/CODEOWNERS so no file or newly created directory in the repository can ever bypass maintainer review.
  • Hardens GitHub Ruleset #26051946 (Protect main branch) to enforce:

equire_code_owner_review: true and
equired_approving_review_count: 1 (blocking all external PRs until @OdevMatheus approves).

  • dismiss_stale_reviews_on_push: true (invalidating approvals if new commits are pushed after review).

equired_status_checks (Validate Branch, Semantic Title & AI Data Hygiene, Audit Subagents & Hybrid Skill Contracts, Node.js 20 NPX Engine & TypeScript QA Gate, Node.js 22 NPX Engine & TypeScript QA Gate).

  • �ypass_actors for Repository Admin (RoleId: 5) so the solo maintainer can merge maintainer-authored PRs once CI checks pass.

Closes #189

Add global wildcard (* @OdevMatheus) ownership in .github/CODEOWNERS so every repository path requires maintainer review, paired with main branch ruleset enforcement for CODEOWNERS approval, stale review dismissal, and required CI status checks.

Closes #189
@github-actions github-actions Bot added the 📄 docs Documentation, ADRs, templates, and guides label Nov 9, 2026
@OdevMatheus
OdevMatheus merged commit a6369fd into main Nov 9, 2026
9 checks passed
@OdevMatheus
OdevMatheus deleted the ci/enforce-codeowners-and-status-checks branch November 9, 2026 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

📄 docs Documentation, ADRs, templates, and guides

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ci(security): enforce codeowners review, stale review dismissal, and required status checks on main

1 participant