Skip to content

Add Symfony 8 support and update CI matrix#374

Merged
monteiro merged 3 commits intomasterfrom
compat/symfony-8.0
Feb 24, 2026
Merged

Add Symfony 8 support and update CI matrix#374
monteiro merged 3 commits intomasterfrom
compat/symfony-8.0

Conversation

@monteiro
Copy link
Collaborator

Convert XML DI config to PHP DSL since XmlFileLoader was removed in
Symfony 8. Bump minimum Symfony version to 5.4 and add ~8.0 constraint.
Update CI to test PHP 8.1–8.5 and drop EOL PHP 7.4/8.0.

Following PR: #373
Thanks to @AdamReece-WebBox

  Convert XML DI config to PHP DSL since XmlFileLoader was removed in
  Symfony 8. Bump minimum Symfony version to 5.4 and add ~8.0 constraint.
  Update CI to test PHP 8.1–8.5 and drop EOL PHP 7.4/8.0.
@monteiro
Copy link
Collaborator Author

@stof do you see any warning from here?

@monteiro monteiro mentioned this pull request Feb 23, 2026
- Remove global SYMFONY_REQUIRE env var from CI (use composer.json constraints)
- Add PHP 7.4 and 8.0 back to CI matrix to match composer.json requirement
- Upgrade actions/checkout from v4 to v6
- Load disable_annotations.yml conditionally for Symfony < 8 only
@monteiro monteiro requested a review from stof February 23, 2026 18:05
@monteiro
Copy link
Collaborator Author

@stof let me know if now it's fine.

Copy link
Contributor

@stof stof left a comment

Choose a reason for hiding this comment

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

The Symfony 8 upgrade looks good to me.

@monteiro monteiro merged commit f227ebd into master Feb 24, 2026
9 checks passed
@stof
Copy link
Contributor

stof commented Feb 24, 2026

@monteiro I suggest tagging a new release with this support.

@monteiro monteiro deleted the compat/symfony-8.0 branch February 24, 2026 14:51
@AdamReece-WebBox
Copy link

Very neat, thanks for getting this over the line. :)

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.

3 participants