Skip to content

PHP requirement for 8.4#15

Open
lphoumpakka wants to merge 1 commit into
masterfrom
fix-php-8-4
Open

PHP requirement for 8.4#15
lphoumpakka wants to merge 1 commit into
masterfrom
fix-php-8-4

Conversation

@lphoumpakka

@lphoumpakka lphoumpakka commented Jun 11, 2026

Copy link
Copy Markdown

This pull request fixes issue #{id}.

It will apply the following changes :


Note

Low Risk
Dependency and version metadata only; runtime blocklist logic is unchanged, though dev/CI may need PHP 8.4+ where Symfony 8.1 packages apply.

Overview
Adds PHP 8.4 to Composer’s platform / require constraint (~8.4.0) so installs and CI can run on 8.4, and refreshes composer.lock so dev tooling resolves under that constraint (notably several Symfony components moving to v8.1 with PHP >= 8.4.1, plus patch bumps for GrumPHP, phplint, security-advisories, and related transitive packages).

Releases 1.0.7 in package.json and the plugin header; no runtime plugin PHP code changes.

Reviewed by Cursor Bugbot for commit 18f31fa. Bugbot is set up for automated code reviews on this repo. Configure here.

@cursor cursor 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.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 18f31fa. Configure here.

Comment thread composer.lock
},
"require": {
"php": ">=8.2",
"php": ">=8.4.1",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Composer lock breaks older PHP

High Severity

The updated composer.lock pins dev tools such as Symfony 8.1 with php >=8.4.1, while composer.json still allows PHP 8.0 through 8.4 and CI runs composer install on 8.0–8.3. Installs from this lock fail on those versions, so the supported matrix no longer matches the committed lock state.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 18f31fa. Configure here.

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