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

symfony ^6.3 || 7 #6007

Closed
wants to merge 13 commits into from
Closed

symfony ^6.3 || 7 #6007

wants to merge 13 commits into from

Conversation

tacman
Copy link
Contributor

@tacman tacman commented Nov 29, 2023

Q A
Branch? main
Tickets Closes #6004
License MIT

Simply allows Symfony 7 to be installed, but there may be an issue with phpspec

@tacman
Copy link
Contributor Author

tacman commented Nov 29, 2023

Well, I see that CI has failed with lots of issues, most relatively small, but I think the complicated part is backwards compatibility (which is why I bumped the requirements to 6.3, since Symfony hasn't necessarily made the signature changes to unsupported versions).

* Revert "Revert "fix: missing eager joins on to-one relationships (api-platform#5992)""

This reverts commit dac49cb.

* test: fix related dummies related tests

* test: duplicate changes in mongoDB documents
@soyuka soyuka mentioned this pull request Nov 29, 2023
@soyuka
Copy link
Member

soyuka commented Nov 29, 2023

I took your commit at #6009 not sure why we would bump to 6.3 I want to see the ci running like this to see if there are things we need to fix

tacman and others added 4 commits November 29, 2023 14:28
Symfony 7 adds a new parameter `$buildDir` to
`WarmableInterface::warmUp()`, so that the method signature of
`CachePoolClearerCacheWarmer::warmUp()` needs to be updated.
@soyuka
Copy link
Member

soyuka commented Nov 29, 2023

mhh weird sorry I pushed on your upstream I didn't wanted to, we'll continue in my PR thanks!

@soyuka soyuka closed this Nov 29, 2023
@tacman tacman deleted the symfony7 branch November 29, 2023 17:39
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.

Support Symfony 7
5 participants