Skip to content

Commit d6b72d3

Browse files
ci: raise the verification job's ceiling to twenty-five minutes
The Linux profile drives the real-browser suite, and that suite launches a browser per case — each case needs a clean extension state, so the launches cannot be shared or parallelised. It no longer fits in the fifteen-minute job ceiling: the base branch's Linux job takes eleven minutes, and this branch adds the real-browser cases on top of it, so the job was killed and the aggregate check failed with it. A ceiling is not a cost — a job that finishes in eleven minutes still finishes in eleven minutes.
1 parent 784ed89 commit d6b72d3

1 file changed

Lines changed: 5 additions & 1 deletion

File tree

‎.github/workflows/ci.yml‎

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,11 @@ jobs:
5151
needs: changes
5252
if: needs.changes.outputs.docs_only == 'false'
5353
name: verify (${{ matrix.os }}, ${{ matrix.node }})
54-
timeout-minutes: 15
54+
# A ceiling, not a cost: a job that finishes in eleven minutes still takes
55+
# eleven minutes. The Linux profile drives the real-browser suite, which
56+
# launches a browser per case because each case needs a clean extension
57+
# state, and that suite does not fit in fifteen minutes any more.
58+
timeout-minutes: 25
5559
strategy:
5660
fail-fast: false
5761
matrix:

0 commit comments

Comments
 (0)