Skip to content

Allow symfony/yaml ^5.4 and ^6.0#2

Merged
mmenozzi merged 1 commit into
php8-compatibilityfrom
widen-symfony-yaml-constraint
Jul 21, 2026
Merged

Allow symfony/yaml ^5.4 and ^6.0#2
mmenozzi merged 1 commit into
php8-compatibilityfrom
widen-symfony-yaml-constraint

Conversation

@mmenozzi

@mmenozzi mmenozzi commented Jul 21, 2026

Copy link
Copy Markdown
Member

Context

A downstream project needs to update openmage/magento-lts to a version that requires symfony/validator ^6.4, which in turn requires symfony/yaml >=5.4. That's incompatible with this fork's current constraint (^3.0 || ^4.0), blocking composer update.

Change

Widen the constraint to ^3.0 || ^4.0 || ^5.4 || ^6.0.

The only usage of Yaml in this fork is Symfony\Component\Yaml\Yaml::parse() (src/DataSet/SymfonyYamlParser.php), a stable API across all these majors.

The branch history shows two earlier attempts to raise this constraint (to ^5.0 and ^6.0), later reverted back to ^3.0 || ^4.0 (commits 1fa1a5e, 5d48fcb, November 2022) — likely due to a conflict with other dependencies at the time. That conflict does not appear to apply anymore in current downstream usage.

The only usage of Yaml in this fork is Symfony\Component\Yaml\Yaml::parse()
(src/DataSet/SymfonyYamlParser.php), an API stable across all these majors.
Needed to unblock newer packages that require symfony/yaml >=5.4.
@mmenozzi
mmenozzi force-pushed the widen-symfony-yaml-constraint branch from 7c3798d to 3b006ad Compare July 21, 2026 10:38
@mmenozzi
mmenozzi merged commit 78a1bca into php8-compatibility Jul 21, 2026
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