Skip to content

Commit 72ab847

Browse files
ci: allow snapshot harness more time
Two hosted snapshot runs completed setup normally and kept passing cases until GitHub cancelled each job at the exact 60-minute ceiling. The suite had reached its late curated Clippy cases, so the cancellation did not identify a compiler or snapshot failure.\n\nRaise only the snapshot shard ceiling to 90 minutes and update the focused workflow contract to prevent an accidental reduction below the newly observed requirement. The harness stages, six-worker parallelism, cleanup policy, and test coverage remain unchanged.\n\nVerified with the focused CI workflow contract, JavaScript syntax checking, and git diff validation. Hosted CI on this branch provides the final full-suite proof.
1 parent 41e2cc8 commit 72ab847

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

.github/workflows/ci.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -250,7 +250,7 @@ jobs:
250250
harness-snapshots:
251251
name: Harness / snapshots
252252
runs-on: ubuntu-latest
253-
timeout-minutes: 60
253+
timeout-minutes: 90
254254
steps:
255255
- name: Checkout
256256
uses: actions/checkout@v5

test/scripts/ci-harness-workflow.test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ const timeout = snapshots.match(/^ timeout-minutes: (\d+)$/m)
1919

2020
assert(timeout, 'snapshot harness job must have an explicit timeout')
2121
assert(
22-
Number(timeout[1]) >= 60,
22+
Number(timeout[1]) >= 90,
2323
`snapshot harness timeout must cover the observed full-suite runtime; found ${timeout[1]} minutes`
2424
)
2525
assert.match(

0 commit comments

Comments
 (0)