search

thecodingmachine / graphqlite-bundle

A Symfony bundle for thecodingmachine/graphqlite.
34 stars 37 forks source link

symfony 6 (requires SF 5.3+) #119

Closed janatjak closed 2 years ago

mistraloz commented 2 years ago

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. This PR will be merged on a new 5.4.x tag.

n-ramdi commented 2 years ago

Hello and thanks for your PR, Any up for this plz?

faizanakram99 commented 2 years ago

@janatjak Thanks for the PR. Are you still working on this ? If not, I can try to pick it up

janatjak commented 2 years ago

I have added new requirements, PHP 7.4+ & SF 5.4+ and updated Travis matrix

@faizanakram99 pls check last commit ;)

swangee commented 2 years ago

Hello, any updates? We would like to update Symfony in our project, but waiting for this PR to be merged.

wollukav commented 2 years ago

Hello! The same for us. It is one of the blockers for migrating to Symfony6. Do you have plans to merge and release it or we should use a fork?

janatjak commented 2 years ago

ping @faizanakram99

please rerun pipelines, thanks 🙏

andrew-demb commented 2 years ago

@devmaslov @moufmouf @nguyenk @npeguin could you help to proceed this PR, please?

faizanakram99 commented 2 years ago

ping @faizanakram99

please rerun pipelines, thanks 🙏

Oh, I'm not a maintainer, just a concerned developer like you