Skip to content

⬆️ drop support for PHP 8.1, add it for 8.6 - #72

Merged
garak merged 1 commit into
masterfrom
drop-php81
Sep 26, 2026
Merged

garak merged 1 commit into
masterfrom
drop-php81

Conversation

@garak

@garak garak commented Sep 25, 2026

Copy link
Copy Markdown
Member

No description provided.

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.

Copilot review overview

🟡 Changes recommended

The CI matrix must retain PHP 8.5 coverage and handle pre-release PHP 8.6 explicitly.

Get a fresh assessment by requesting another Copilot review.

Review effort: Lite
Findings: 1 Medium severity

Open (1)
What changed in this PR

Updates PHP support and dependency constraints while refreshing Symfony/PHP CI coverage.

Changes:

  • Raises minimum PHP support to 8.2.
  • Updates runtime and development dependencies.
  • Revises the Symfony/PHP compatibility matrix.
  • Retains stable PHP 8.5 coverage alongside any PHP 8.6 testing.
File Summary
composer.json Updates runtime and development dependency requirements.
.github/​workflows/​build.yaml Refreshes CI versions; PHP 8.6 should not replace stable PHP 8.5 coverage.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread .github/workflows/build.yaml
@sonarqubecloud

Copy link
Copy Markdown

@garak
garak merged commit 45fdfa0 into master Sep 26, 2026
10 of 12 checks passed
@garak
garak deleted the drop-php81 branch September 26, 2026 06:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants