Skip to content

docs(process): name the branch that exists and the checks that run #23

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
max578 merged 1 commit into from
Sep 9, 2026
+8 −8
Merged

docs(process): name the branch that exists and the checks that run #23

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions CONTRIBUTING.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,8 +26,8 @@ source this repository did not author.
## Review

Every pull request is reviewed under `REVIEW.md` and needs one approving
review from an author who did not write it. Branch protection on
`acir-package` and `main` enforces this and the required checks.
review from an author who did not write it. Branch protection on `main`
enforces this and the required checks.

## Reporting

Expand Down
8 changes: 4 additions & 4 deletions REVIEW.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,9 +1,9 @@
# Review policy

This applies to every pull request into `acir-package` and `main`, and to
every reviewer, human or automated. A review is a list of findings ranked by
severity. Approval is a separate act, by an author who did not write the
change, and branch protection requires it before a merge.
This applies to every pull request into `main`, and to every reviewer, human
or automated. A review is a list of findings ranked by severity. Approval is
a separate act, by an author who did not write the change, and branch
protection requires it before a merge.

## Three passes

Expand Down
4 changes: 2 additions & 2 deletions dev/PROCESS.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ this process applies from 2026-09-02.

## The four gates

Every pull request into `acir-package` carries these, and CI runs them again.
Every pull request into `main` carries these, and CI runs them again.

1. **Numbers.** Every fixture in `acir/inst/evidence/register.csv` at its
tolerance class, and the package's stored outputs on the reference records
Expand Down Expand Up @@ -71,7 +71,7 @@ and is a review finding in its own right.
| Feedback loop before reporting done | the four gates, run locally and in CI | in use |
| Continuous evaluation of agent configuration | not applicable to the repository | not used |
| Review loop under a written policy | `REVIEW.md`; approval by a human author is required | in use |
| Hooks as approval gates | branch protection on `acir-package` and `main`: pull request required, one approving review, nine required checks, no force push | in use, enforced by the host |
| Hooks as approval gates | branch protection on `main`: pull request required, one approving review, ten required checks, no force push | in use, enforced by the host |
| An agent inside CI/CD | not used; CI is deterministic | not used |
| Control bands in maintenance | `bench.yaml` compares every run with `baseline.csv`; a breach is triaged by the authors and re-enters as an intent | in use, with a manual response |
| Recurring security scans | not used; the package opens no network connection (`SECURITY.md`) | not used |
Expand Down
Loading