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

Add support for symfony 7.0 #501

Closed
allan-simon opened this issue Dec 6, 2023 · 6 comments
Closed

Add support for symfony 7.0 #501

allan-simon opened this issue Dec 6, 2023 · 6 comments

Comments

@allan-simon
Copy link

tell me if there's a way to help you on that :)

@bocharsky-bw
Copy link
Member

I think the first step would be to allow ... || ^7.0 in composer.json for all Symfony related packages and then fix broken tests.

Do you want to create a PR for this?

@webspin
Copy link

webspin commented May 27, 2024

Hello,

What is the status of this issue?

Kind regards

@antiftw
Copy link

antiftw commented Jun 9, 2024

There's an open PR already.

However it's stuck on this.

I've started assisting on the one remaining dependency that's still open. We're closing in on a solution.

Keep an eye on those for updates.

@OliviergSingulart
Copy link

I see that the MR on the extractor is now merge, how can we help move it along for an official release here too?

This bundle is my last dependency preventing me from upgrading to 7.1 (along with php-translation/loco-adapter but there's an MR on that one I could use)

@bocharsky-bw
Copy link
Member

We had a PR for Sf 7 support here: #502 - but recently author closed it, I suppose because stuck at fixing related tests.

If you could give it a try to create a new PR with the necessary changes - I would be happy to help reviewing and merging it. Basically, we need to allow Sf 7 in composer.json and fix related tests if any failed

@bocharsky-bw
Copy link
Member

Closed with #509

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 a pull request may close this issue.

5 participants