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

Make Symfony 6.3 the minimum version #6335

Merged
merged 5 commits into from Aug 26, 2023
Merged

Conversation

leofeyer
Copy link
Member

No description provided.

@leofeyer leofeyer added this to the 5.3 milestone Aug 25, 2023
@leofeyer leofeyer self-assigned this Aug 25, 2023
@leofeyer leofeyer requested a review from a team August 25, 2023 11:34
@leofeyer leofeyer changed the title Make Symfony 6.2 the minimum version Make Symfony 6.3 the minimum version Aug 25, 2023
ausi
ausi previously approved these changes Aug 25, 2023
Copy link
Member

@ausi ausi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Didn’t manually test the changes of ModelArgumentResolver.php

composer.json Outdated Show resolved Hide resolved
composer.json Outdated Show resolved Hide resolved
leofeyer and others added 2 commits August 25, 2023 15:13
Co-authored-by: Martin Auswöger <martin@auswoeger.com>
Co-authored-by: Martin Auswöger <martin@auswoeger.com>
@leofeyer leofeyer requested a review from ausi August 25, 2023 13:14
@leofeyer
Copy link
Member Author

leofeyer commented Aug 25, 2023

As @aschempp pointed out in Slack, this will make it impossible for an extension to be compatible with both Contao 4.13 and 5.3, because Contao 4.13 is only compatible with Symfony 5. There are two possible solutions:

  1. Make Contao 4.13 compatible with Symfony 6.
  2. Keep Contao 5.3 compatible with Symfony 5.

And I really hope it will not be option 2! 🤞

@ausi
Copy link
Member

ausi commented Aug 25, 2023

As @aschempp pointed out in Slack, this will make it impossible for an extension to be compatible with both Contao 4.13 and 5.3, because Contao 4.13 is only compatible with Symfony 5.

I don’t see why that should be the case. The extension simply has to use "symfony/…": "^5.4 || ^6.4" that should be straightforward and not a problem at all.

@leofeyer leofeyer merged commit 8e54a89 into contao:5.x Aug 26, 2023
16 checks passed
@leofeyer leofeyer deleted the feature/symfony-6 branch August 26, 2023 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants