Skip to content

ci: mark 'Coq build — formal/_CoqProject' as REQUIRED in main ruleset (owner action) #244

Closed

Description

@hyperpolymath

Summary

`coq-build.yml` shipped via #231 (closed #227). EACCES container fix is in flight via #236. Print Assumptions audit expanded to all four Phase D theorems via #243.

Remaining gap: the `Base` branch ruleset on `main` does not include the new Coq build check as a required status check. Until it does, the workflow is informational — `admin-merge` can still land a broken main even after #236 + #243 are in.

Verification

```bash
gh api repos/hyperpolymath/ephapax/rulesets/14285235 --jq '.rules[].type'
```

Returns:
```
deletion
non_fast_forward
required_signatures
pull_request
```

No `required_status_checks` rule. The Coq build runs (or will, post-#236) but its result is not enforced.

Acceptance

  • Add `required_status_checks` rule to the `Base` ruleset on `main` listing the context `Coq build — formal/_CoqProject` (and the GitHub Actions integration ID).
  • Verify by attempting to admin-merge a no-op PR while the check is red — should be blocked.

Suggested API shape

```bash

First confirm the Actions integration ID for this repo:

gh api repos/hyperpolymath/ephapax/installation --jq '.app_id'

Then PATCH the ruleset to append the required_status_checks rule.

See:

https://docs.github.com/en/rest/repos/rules#update-a-repository-ruleset

```

Why this matters

The standing rule [[feedback_proof_pr_build_oracle_is_only_truth]] is the only safety net while the gate is informational. Marking the check REQUIRED moves enforcement from human discipline to GitHub's merge guard — the original motivation behind #227.

Refs

🤖 Generated with Claude Code

Activity

  1. hyperpolymath commented on May 30, 2026

    @hyperpolymath
    OwnerAuthor

    Done — required_status_checks rule added to Base ruleset (id 14285235).

    $ gh api repos/hyperpolymath/ephapax/rulesets/14285235 --jq '.rules[].type'
    deletion
    non_fast_forward
    required_signatures
    pull_request
    required_status_checks
    
    $ gh api repos/hyperpolymath/ephapax/rulesets/14285235 \
        --jq '.rules[] | select(.type=="required_status_checks") | .parameters.required_status_checks'
    [{"context":"Coq build — formal/_CoqProject","integration_id":15368}]
    

    integration_id: 15368 = github-actions app (confirmed from existing check-runs on main). The gate is now HARD: any PR touching formal/** or .github/workflows/coq-build.yml must pass Coq build — formal/_CoqProject before merge; admin-merge requires explicit bypass.

    Closing — acceptance checkbox 1 satisfied. Checkbox 2 (admin-merge-while-red empirical test) deferred to natural confirmation the next time the workflow goes red on a PR.

  2. added 2 commits that reference this issue on May 30, 2026
    abc58b1
    48d996f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

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