Skip to content
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

Support Symfony 7 #1930

Closed
wants to merge 2 commits into from
Closed

Support Symfony 7 #1930

wants to merge 2 commits into from

Conversation

endroid
Copy link

@endroid endroid commented Nov 7, 2023

Add support for Symfony 7

@evertharmeling
Copy link
Contributor

evertharmeling commented Nov 20, 2023

LGTM, probably #1931 should be merged together

@endroid
Copy link
Author

endroid commented Nov 21, 2023

Yes or merge them both and do a final fix if necessary.

@l-you l-you mentioned this pull request Dec 3, 2023
@l-you
Copy link
Contributor

l-you commented Dec 3, 2023

@arghav @alanbem @xabbuh @ddeboer @merk
Does anyone know where the contributors with approval access are?
Symfony 7 support is importnat update.

@xabbuh
Copy link
Member

xabbuh commented Dec 4, 2023

I don't have any merge access here

@XWB
Copy link
Member

XWB commented Dec 4, 2023

I'd be happy to merge if you fix the pipeline.

@l-you l-you mentioned this pull request Dec 4, 2023
@endroid
Copy link
Author

endroid commented Dec 5, 2023

I see someone else has also started working on this in the meanwhile so I'm closing this one. No use in multiple people building the same thing :)

@endroid endroid closed this Dec 5, 2023
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.

5 participants