• Fix YAML indentation for sleep command in integration tests run block
• Add missing cache step ID (cache-php-windows) in Windows integration test job
• Fix artifact name collision by including matrix variables in artifact name
• Fix typo 'Archive test results results' -> 'Archive test results'
• Add timeout-minutes to all 12 jobs to prevent runaway builds

This Pull Request improves CI workflow stability and fixes multiple configuration issues.

• I read the Generative AI policy and my contribution is either not created with the help of AI or is compatible with the policy and GNU/GPL 2 or later.

Summary of Changes

This PR fixes multiple issues in .github/workflows/ci.yml related to CI reliability, caching behavior, artifact uploads, and workflow safety.

1. Fix YAML indentation in integration tests

The sleep 3 command inside a YAML literal block (|) used inconsistent indentation (9 spaces instead of 10), which could lead to readability issues and potential parsing inconsistencies.

2. Add missing cache step ID in Windows integration tests

The tests-integration-windows job referenced:

steps.cache-php-windows.outputs.cache-hit

However, the cache restore step was missing:

id: cache-php-windows

This caused Composer dependencies to reinstall on every run even when cache restoration succeeded.

3. Fix artifact name collision in system tests

The tests-system matrix job previously used a static artifact name:

system-test-output

Since actions/upload-artifact@v4 requires unique artifact names, parallel matrix executions (6 DB configs × 2 browsers) caused upload conflicts.

Artifact names now include matrix variables:

system-test-output-${{ matrix.config.test_group }}-${{ matrix.browser }}

Also fixes typo: Archive test results results → Archive test results

4. Add timeout protection to all jobs

No jobs defined timeout-minutes, defaulting to GitHub's 360-minute limit.

Reasonable timeout limits were added to prevent runaway builds and unnecessary runner consumption:

Testing Instructions

1. Confirm workflow YAML validation passes.
2. Run CI on a test branch.
3. Verify:
   • System test artifacts upload successfully for all matrix runs.
   • Windows integration tests skip Composer install when cache is hit.
   • Jobs terminate correctly when exceeding timeout limits.

Actual result BEFORE applying this Pull Request

• Inconsistent YAML indentation
• Composer cache unused on Windows integration tests
• Artifact upload collisions in matrix jobs
• Jobs could run up to 6 hours

Expected result AFTER applying this Pull Request

• Consistent YAML formatting
• Functional Composer caching
• Unique artifact uploads per matrix execution
• Safe execution time limits

Link to documentations

Please select:

• Documentation link for guide.joomla.org:

• No documentation changes for guide.joomla.org needed

• Pull Request link for manual.joomla.org:

• No documentation changes for manual.joomla.org needed