You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
freeshard loop: ci_required needs a real per-repo CI signal #95
Whole-branch review Important finding. The no-CI-handoff fix uses ci_required = verify_profile(repo) != "none" as a proxy. But verify_profile=="none" means "no test suite", not "no CI" — a repo can have no local tests yet still run GitHub Actions (link-check, build, spell-check).
If documentation actually runs CI, the loop wrongly skips the CI wait and hands off immediately — the exact behavior the gate should prevent. documentation#2 itself adds a CI workflow, flipping docs from no-CI → has-CI on merge.
Fix: a real per-repo ci_required signal decoupled from _NO_TESTS (detect .github/workflows presence, or an explicit config set). Depends on each non-core repo’s actual CI setup.
Resolved in PR #97 (commit 84ec6f5): ci_required is now derived live from has_ci_workflows() (real .github/workflows presence), replacing the verify_profile=="none" proxy. Defaults to True on API error (safer to wait).
Activity
max-tet commentedon Jun 29, 2026
I would like to have detection of
.github/workflowsas the one single signal if that is possible.ClaydeCode commentedon Jun 29, 2026
Resolved in PR #97 (commit 84ec6f5):
ci_requiredis now derived live fromhas_ci_workflows()(real.github/workflowspresence), replacing theverify_profile=="none"proxy. Defaults toTrueon API error (safer to wait).