-
-
Notifications
You must be signed in to change notification settings - Fork 1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Allow Symfony 5 #2970
Allow Symfony 5 #2970
Conversation
Wait , don't merge yet :) |
PR is green and ready. |
if (trait_exists(MailerAssertionsTrait::class)) { | ||
$loader->load(function (ContainerBuilder $container) { | ||
$container->loadFromExtension('twig', [ | ||
'exception_controller' => null, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if we can limit this config to 4.4.* strictly, could we remove then this (symfony/symfony#34062) workaround ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think symfony/symfony#34062 is important on its own. The less painful the upgrade path the better!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great! Nicolas, thanks a lot for working on this 🙏 It was on my TODO list but I couldn't find time for it yet. Thanks to Yonel too for reviewing!
This PR was merged into the 2.x branch. Discussion ---------- Allow Symfony 5 Related to doctrine/DoctrineFixturesBundle#297 Commits ------- 32b8917 Allow Symfony 5
Related to doctrine/DoctrineFixturesBundle#297