Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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: 3 additions & 1 deletion .github/workflows/cora-review.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,9 @@ jobs:
cora-review:
name: Cora Review
runs-on: ubuntu-latest
timeout-minutes: 10
# PRs of 850+ lines need ~16 min of review; 10 min cancelled them
# mid-review with no verdict.
timeout-minutes: 30
steps:
- name: Checkout PR head
uses: actions/checkout@v7
Expand Down
6 changes: 3 additions & 3 deletions AGENTS.md
Original file line number Diff line number Diff line change
Expand Up @@ -40,9 +40,9 @@ continuity, infra facts.
verdict in the PR comments ("✅ No issues found" or concrete findings —
"Review could not complete" is a blocker), code-scanning alerts
investigated (real → fix with regression test; FP → dismiss with
evidence), CI auto-fix max 3 attempts then escalate. Large change-sets
(~40+ files) time out Cora Review (10-min limit) — split into ~15–20-file
PRs.
evidence), CI auto-fix max 3 attempts then escalate. The Cora Review job
has a 30-min limit; large change-sets (~40+ files or ~850+ lines) can still
time out — split into ~15–20-file PRs.

## 3. Mandatory pre-commit checks

Expand Down
5 changes: 2 additions & 3 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -32,9 +32,8 @@ workflow; the ecosystem-wide standards it follows live in the team knowledge roo

## Pull requests

- One logical change per PR; large change-sets (~40+ files) deterministically
time out the Cora Review job (10-minute hard limit) — split into ~15–20-file
PRs.
- One logical change per PR; large change-sets (~40+ files) can time out the
Cora Review job (30-minute limit) — split into ~15–20-file PRs.
- Body sections, in order: **What / Why / Changes / Testing**. A PR without
them fails the PR Description check.
- Base every PR on `develop`. PRs into `main` are only accepted from
Expand Down
Loading