8ad5fecf3a
Records why main is protected with CI status checks only — no required approvals, no required code-owner review. Public exclusion comes from repo WRITE ACCESS, not a review rule; a self-approval requirement on a single-identity repo only creates an unsatisfiable deadlock that forces --admin on every merge (cf #772). - ADR-0221: context (the deadlock), the misconception corrected, the decision, an explicit 'do not re-add human-review gates' guard for future agents, the applied gh-api change, and the break-glass log. - CODEOWNERS: comment corrected to ADVISORY-ONLY (require_code_owner_reviews is now false); keeps * @AssetOverflow for ownership/auto-request, not gating. Applied 2026-06-15: required_approving_review_count 1->0, require_code_owner_reviews true->false; required status checks unchanged. This PR is the first through the fixed normal path — its clean, approval-free, no-admin merge is the proof.
14 lines
787 B
Text
14 lines
787 B
Text
# Code ownership for AssetOverflow/core — ADVISORY ONLY (see ADR-0221).
|
|
#
|
|
# This repository is public with a single maintainer. Public exclusion is
|
|
# provided by repository WRITE ACCESS — non-collaborators can open PRs but can
|
|
# never merge — NOT by a review requirement. Per ADR-0221, branch protection on
|
|
# `main` is required-status-checks-only: required_approving_review_count = 0 and
|
|
# require_code_owner_reviews = false. This file therefore only auto-requests
|
|
# review / assigns ownership; it does NOT gate merges.
|
|
#
|
|
# Do NOT re-enable required code-owner review on this solo-maintainer repo: every
|
|
# PR is authored by the only owner, and GitHub forbids self-approval, so it
|
|
# creates an unsatisfiable deadlock that forces --admin on every merge (ADR-0221).
|
|
|
|
* @AssetOverflow
|