Commit 2eccaff
authored
Raise pr-review job timeout to 30 minutes (#108)
## Why
The reviewer model is now `claude-opus-5`, which runs longer per review
than opus-4-8 (and the automatic cybersecurity→opus-4.8 safety fallback
can stack a second run into the same job). On the ductone side the same
model bump drove judge PR-review timeouts from 0/809 runs to 20/191
(10.5%) against a 15-minute cap — cancelled mid-review, surfacing as a
"did not finish in time … not a verdict" notice rather than a real
verdict. This raises the caps here before the same regression shows up.
## What this changes
Raises `timeout-minutes` to 30 on both PR-review reusable-workflow jobs:
`pr-review.yaml` (15→30) and `general-pr-review.yaml` (20→30).
Trade-off: a longer-running or wedged review now uses up to 30
runner-minutes before being cut off.1 parent 1ee2337 commit 2eccaff
3 files changed
Lines changed: 3 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
87 | 87 | | |
88 | 88 | | |
89 | 89 | | |
90 | | - | |
| 90 | + | |
91 | 91 | | |
92 | 92 | | |
93 | 93 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
42 | 42 | | |
43 | 43 | | |
44 | 44 | | |
45 | | - | |
| 45 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
48 | 48 | | |
49 | 49 | | |
50 | 50 | | |
51 | | - | |
| 51 | + | |
0 commit comments