Skip to content

Update PHP version requirement to include 8.5 - #1492

Open
KorvinSzanto wants to merge 2 commits into
Lctrs:masterfrom
KorvinSzanto:patch-1
Open

Update PHP version requirement to include 8.5#1492
KorvinSzanto wants to merge 2 commits into
Lctrs:masterfrom
KorvinSzanto:patch-1

Conversation

@KorvinSzanto

@KorvinSzanto KorvinSzanto commented Jan 28, 2026

Copy link
Copy Markdown

Summary by CodeRabbit

  • Chores

    • Extended PHP compatibility to include PHP 8.5.
    • Added new required status checks to branch protection for PHP 8.5.
  • Tests

    • Added PHP 8.5 to the CI test matrix.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai

coderabbitai Bot commented Jan 28, 2026

Copy link
Copy Markdown
📝 Walkthrough

Walkthrough

Updated project to add PHP 8.5 support: extended composer PHP requirement and added PHP 8.5 entries to CI configuration and required status checks.

Changes

Cohort / File(s) Summary
PHP Version Constraint
composer.json
Extended PHP version requirement to include ~8.5 alongside existing `~8.2.27
CI Status Checks
.github/settings.yml
Added two required status check contexts for PHP 8.5: Tests (8.5, locked) and Tests (8.5, lowest).
CI Workflow Matrix
.github/workflows/integrate.yaml
Added PHP 8.5 to the php-version testing matrix for CI runs.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 I nibble code beneath the sky,
A new eight-point-five hops by,
Composer changed and tests now run,
Bright PHP dawn—what fun! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately and concisely describes the primary change across all modified files: adding PHP 8.5 support to composer.json, CI configuration, and branch protection rules.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@coderabbitai coderabbitai Bot 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.

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In `@composer.json`:
- Line 28: Update the vimeo/psalm constraint in composer.json from the current
^6.13.1 to ^6.14.0 so Psalm can run on PHP 8.5; change the require-dev entry for
"vimeo/psalm" to "^6.14.0" and then update the lockfile by running composer
require --dev vimeo/psalm:^6.14.0 or composer update vimeo/psalm
--with-dependencies to ensure composer.lock reflects the new constraint.

Comment thread composer.json
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