Skip to content

CI: modernize GitHub Actions (PHP 8.2–8.5) - #11

Merged
slashrsm merged 1 commit into
masterfrom
ci/phase0-modernize-actions
Jul 22, 2026
Merged

CI: modernize GitHub Actions (PHP 8.2–8.5)#11
slashrsm merged 1 commit into
masterfrom
ci/phase0-modernize-actions

Conversation

@slashrsm

Copy link
Copy Markdown
Contributor

Summary

Phase 0 housekeeping for green, maintainable CI.

  • Bump actions/checkout and actions/cache to v4
  • Matrix: PHP 8.2–8.5
  • Remove Scrutinizer/ocular upload

Part of PortPHP org CI modernization (Phase 0).

Phase 0 housekeeping:
- Upgrade actions/checkout and actions/cache to v4
- Test on PHP 8.2–8.5
- Remove Scrutinizer coverage upload
- Fail the suite on test errors (no continue-on-error)
@slashrsm
slashrsm merged commit 70cd9bd into master Jul 22, 2026
4 of 5 checks passed
@slashrsm
slashrsm deleted the ci/phase0-modernize-actions branch July 22, 2026 10:58
@slashrsm
slashrsm requested a review from Copilot July 22, 2026 11:07

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.

Pull request overview

Modernizes the project’s CI workflow to use current GitHub Actions versions and a newer PHP test matrix, aligning with the PortPHP org “Phase 0” CI modernization effort.

Changes:

  • Upgraded actions/checkout and actions/cache to v4.
  • Updated the test matrix to run on PHP 8.2–8.5 (and disabled fail-fast).
  • Removed Scrutinizer/ocular coverage upload and standardized on running PHPUnit without coverage.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +32 to +33
- name: Validate composer.json
run: composer validate --strict --no-check-lock
Comment on lines 38 to +41
path: vendor
key: ${{ runner.os }}-composer-${{ matrix.composer-prefer }}$-${{ hashFiles('**/composer.lock') }}
key: ${{ runner.os }}-composer-${{ matrix.php-versions }}-${{ hashFiles('**/composer.json') }}
restore-keys: |
${{ runner.os }}-composer-${{ matrix.composer-prefer }}-
${{ runner.os }}-composer-${{ matrix.php-versions }}-
@slashrsm slashrsm mentioned this pull request Jul 22, 2026
2 tasks
slashrsm added a commit that referenced this pull request Jul 22, 2026
Declare php ^8.2, lock-aware validate/install, cache key includes lock, --no-interaction.
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