Skip to content

Symfony 7/8 Fixes & Compatibility#505

Open
dsoriano wants to merge 9 commits intolexik:masterfrom
dsoriano:sf8-compatibility
Open

Symfony 7/8 Fixes & Compatibility#505
dsoriano wants to merge 9 commits intolexik:masterfrom
dsoriano:sf8-compatibility

Conversation

@dsoriano
Copy link
Copy Markdown
Contributor

@dsoriano dsoriano commented Mar 15, 2026

This pull request aim to improve the whole bundle compatibility with Symfony 7 / 8

  • Move the LexikTranslator to a Symfony Translator decorator
  • As a decorator, the bundle now use most of the Symfony Translator configuration (but no BC Break with olds configs)
  • Fix all level 5 PHPStan errors (~80)
  • Remove or upgrade deprecated code
  • Fix tests
  • Clean Code with CS Fixer
  • Update configuration doc
  • Fix Docker
  • Fix a lot of PHPStorm / PHP Inspections Code analysis reports (but not all at the moment 🥵)

✅ Unit tests OK
✅ Tested on a real project

There is still a lot of work to do but this PR is a good start !

Fix #500 and #490

@dsoriano dsoriano marked this pull request as ready for review March 15, 2026 21:37
@dsoriano
Copy link
Copy Markdown
Contributor Author

@bartmcleod hi !
After a long and hard work, the PR is finally ready and is fully compatible with Symfony 7 and Symfony 8 👍!

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.

Cannot extend final class Translator

1 participant