Skip to content

ci: raise nightly coverage-job timeouts for xdebug branch coverage (#624 S9)#634

Merged
CybotTM merged 1 commit into
mainfrom
fix/nightly-coverage-timeouts
Jul 15, 2026
Merged

ci: raise nightly coverage-job timeouts for xdebug branch coverage (#624 S9)#634
CybotTM merged 1 commit into
mainfrom
fix/nightly-coverage-timeouts

Conversation

@CybotTM

@CybotTM CybotTM commented Jul 15, 2026

Copy link
Copy Markdown
Member

Follow-up to #633 (S9). The nightly / workflow_dispatch coverage run enables xdebug branch coverage, which is ~6× slower than a plain run — the workflow_dispatch verification showed the unit suite exceeding its 10-min timeout (cancelled at 558s), integration at ~703s.

Raise test-unit 10→30 min and test-integration 15→25 min. The PR/push path skips coverage (XDEBUG_MODE=off) and finishes in ~90s, so these timeouts only bind the nightly coverage run.

Refs #624.

 S9)

The nightly / workflow_dispatch coverage run enables xdebug branch coverage,
which is ~6x slower than a plain run — the unit suite exceeded its 10-min
timeout (cancelled at 558s). Raise test-unit 10->30 and test-integration 15->25.
The PR path skips coverage entirely, so it is unaffected (finishes in ~90s).

Signed-off-by: Sebastian Mendel <info@sebastianmendel.de>
Copilot AI review requested due to automatic review settings July 15, 2026 11:31
@gemini-code-assist

Copy link
Copy Markdown

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@sonarqubecloud

Copy link
Copy Markdown

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@CybotTM
CybotTM merged commit 667937a into main Jul 15, 2026
17 of 18 checks passed
@CybotTM
CybotTM deleted the fix/nightly-coverage-timeouts branch July 15, 2026 12:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants