Skip to content

Drop PHP 8.1, upgrade Saloon to v4 and Pest to v3#12

Merged
craigpotter merged 1 commit into
mainfrom
update-min-php-and-deps
Mar 26, 2026
Merged

Drop PHP 8.1, upgrade Saloon to v4 and Pest to v3#12
craigpotter merged 1 commit into
mainfrom
update-min-php-and-deps

Conversation

@craigpotter

@craigpotter craigpotter commented Mar 26, 2026

Copy link
Copy Markdown
Owner

Summary

  • PHP: Bump minimum from ^8.1 to ^8.2 (PHP 8.1 is EOL)
  • saloonphp/saloon: Upgrade from ^3.0 to ^4.0 (addresses critical CVE, supersedes Chore | Upgrade to Saloon v4 #11)
  • pestphp/pest: Upgrade from ^2.0 to ^3.0
  • GitHub Actions: Bump actions/checkout from v3 to v4, drop PHP 8.1 from test matrix
  • Test fixtures: Rename to use slashes/hyphens instead of dots (Saloon v4 stricter validation)

Test plan

  • All 7 tests pass locally with updated dependencies
  • CI passes on PHP 8.2, 8.3, 8.4

PHP 8.1 is EOL — bump minimum to 8.2. Upgrade saloonphp/saloon
from v3 to v4 (CVE fix) and pestphp/pest from v2 to v3. Rename
test fixtures to use slashes/hyphens instead of dots to comply
with Saloon v4's stricter fixture name validation. Bump GitHub
Actions checkout to v4.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

@craigpotter craigpotter left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

SR ✅

@craigpotter
craigpotter merged commit ecde2d7 into main Mar 26, 2026
13 checks passed
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.

1 participant