-
Notifications
You must be signed in to change notification settings - Fork 42
symfony 6 (requires SF 5.3+) #119
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
Conversation
Thanks for your PR but some tests failed. I'm not sure of why but if you need, you can remove some support (php7.2 and 7.3) and can remove too (if required or better) support for symfony 5.3 (just keep ^5.4 | ^6). If you change the CI, please try to add one for php 8.1. |
Hello and thanks for your PR, |
@janatjak Thanks for the PR. Are you still working on this ? If not, I can try to pick it up |
cleanup travis matrix
I have added new requirements, PHP 7.4+ & SF 5.4+ and updated Travis matrix @faizanakram99 pls check last commit ;) |
Hello, any updates? We would like to update Symfony in our project, but waiting for this PR to be merged. |
Hello! |
ping @faizanakram99 please rerun pipelines, thanks 🙏 |
@devmaslov @moufmouf @nguyenk @npeguin could you help to proceed this PR, please? |
Oh, I'm not a maintainer, just a concerned developer like you |
No description provided.