Skip to content

Conversation

@gilzow
Copy link
Contributor

@gilzow gilzow commented Nov 26, 2025

  • updates composer.json
    • require php 7.4 or greater
    • UpdatesPHPUnit to 9.6 (last version to support PHP 7.4, supports up to 8.4)
  • updates github workflow
    • remove php 7.2 from matrix
    • adds php 8.1, 8.2, 8.3 and 8.4
  • updates README
    • reflect name change from Platform.sh to Upsun Fixed
    • Adds Requirement section to indicate PHP 74 or greater is now required

…6, updates github workflow to remove php72, adds php 81, 82, 83 and 84 to github workflow, updates README to reflect name change and requirement of PHP 74 or greater
@gilzow gilzow requested a review from Copilot November 26, 2025 16:34
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the minimum PHP version requirement from 7.2 to 7.4 and modernizes the project's branding and documentation to reflect the Platform.sh to Upsun Fixed transition.

Key Changes:

  • Updates minimum PHP version to 7.4 and PHPUnit to version 9.6 for broader PHP 8.x compatibility
  • Expands CI testing matrix to include PHP versions 8.1 through 8.4
  • Rebrands documentation from Platform.sh to Upsun Fixed throughout the README

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
composer.json Updates PHP requirement to >=7.4 and PHPUnit to ^9.6 for compatibility with modern PHP versions
.github/workflows/quality-assurance.yaml Removes PHP 7.2 and adds PHP 8.1-8.4 to the CI test matrix
README.md Rebrands from Platform.sh to Upsun Fixed and adds Requirements section documenting PHP 7.4+ requirement

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

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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