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

chore: support symfony 7 #6009

Merged
merged 3 commits into from
Nov 30, 2023
Merged

chore: support symfony 7 #6009

merged 3 commits into from
Nov 30, 2023

Conversation

@soyuka soyuka mentioned this pull request Nov 29, 2023
@soyuka soyuka changed the base branch from main to 3.1 November 29, 2023 13:26
@tacman
Copy link
Contributor

tacman commented Nov 29, 2023

Thanks. This is a bit out of my league, so I'll continue to test and cheer from the sidelines.

API Platform is amazing. Thanks for releasing and supporting it!

@soyuka
Copy link
Member Author

soyuka commented Nov 29, 2023

@tacman I think that upgrading the minor version of symfony would require us to bump a minor version as well. API Platform 3.4 will be out in February we will think about upgrading symfony to 6.4 as a minimal requirement at this moment!

tacman and others added 2 commits November 30, 2023 10:11
Symfony 7 adds a new parameter `$buildDir` to
`WarmableInterface::warmUp()`, so that the method signature of
`CachePoolClearerCacheWarmer::warmUp()` needs to be updated.
@soyuka soyuka merged commit 49b412e into api-platform:3.1 Nov 30, 2023
25 of 28 checks passed
@tacman
Copy link
Contributor

tacman commented Nov 30, 2023

Sorry for the dumb question, but how can I test this?

composer req api-platform/core:dev-symfony7

Isn't right, nor dev-main.

@paullallier
Copy link
Contributor

@tacman: It's moot now, since it's released in 3.1.27, but I think you wanted:

composer require api-platform/core:3.1.x-dev

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