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
Drop Symfony support below 4.4 #643
Conversation
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.
The RequestListenerTest::testHandleKernelRequestEvent()
method is still using a data provider that was meant for Symfony < 4.4
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.
There are still things to delete: in the SubRequestListenerTest
class some of the data providers are still splitted per-Symfony version, and probably we can also remove the KernelEventForwardCompatibilityTrait
trait. I suggest to look for occurences of the version_compare
or Kernel::VERSION
keywords
I took care of the rest. |
I'm opening this as a draft since it may require some discussion (or dropping some pieces of code no longer used).3.4 is unsupported since last November: https://symfony.com/releases/3.4
Everything between that and 4.4 (since it's an LTS) is unsupported too.
Hence, dropping the support now makes sense. This PR is targeting
develop
, so it would land in the next minor (4.4 as of now).