Skip to content

Fix 'Main guard' ruleset properly (currently unblocked via admin bypass) #78

@fmasi

Description

@fmasi

Context

The main ruleset "Main guard" requires checks that aren't wired up, and had an empty bypass list — which blocked all merges to main, including the owner's. To ship PRs #76/#77 on 2026-06-09 we added a Repository-Admin bypass actor (bypass_mode: always) as a stopgap. Rules remain active for any non-admin.

The actual problem

"Main guard" requires, but nothing satisfies:

  • code_scanning (CodeQL, high/error thresholds) — CodeQL is not-configured on this repo
  • code_quality
  • copilot_code_review — needs a GitHub Copilot subscription to run
  • pull_request (0 approvals, but requires review-thread resolution)

Note: this is not a billing/Pro issue — the repo is public, so rulesets + CodeQL are free. The checks simply never run.

Proper fix (pick one)

  • Enable CodeQL default setup (Settings → Code security → Code scanning → Default setup) so the code_scanning check actually runs and passes — then it's a real gate, not a lockout.
  • Decide on code_quality + copilot_code_review — keep only if the tooling is available to run them (Copilot review needs an active Copilot sub); otherwise remove those rules.
  • Once the required checks run green, reconsider/remove the admin bypass if you want strict enforcement for yourself too.

Until then, the admin bypass keeps main mergeable.

Filed with Claude Code.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions