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

Drop Symfony 4.4 support #735

Merged
merged 2 commits into from
Jan 5, 2023
Merged

Drop Symfony 4.4 support #735

merged 2 commits into from
Jan 5, 2023

Conversation

TheMilek
Copy link
Member

@TheMilek TheMilek commented Dec 14, 2022

We will no more support Symfony 4.4

@TheMilek TheMilek requested a review from a team as a code owner December 14, 2022 11:02
@TheMilek TheMilek force-pushed the drop-symfony-4.4 branch 3 times, most recently from db7282e to 21615b3 Compare January 4, 2023 10:30
@TheMilek TheMilek changed the title Drop Symfony 4.4 Drop Symfony 4.4, 5.2 and 5.3 support Jan 4, 2023
@TheMilek TheMilek force-pushed the drop-symfony-4.4 branch 4 times, most recently from f8f393f to 4a22c5d Compare January 4, 2023 11:07
@TheMilek TheMilek force-pushed the drop-symfony-4.4 branch 4 times, most recently from 2db2dd8 to 6202788 Compare January 4, 2023 11:49
@TheMilek TheMilek changed the title Drop Symfony 4.4, 5.2 and 5.3 support Drop Symfony 4.4 support Jan 4, 2023
mysql: [5.7, 8.0]
symfony: [^4.4, ~5.2.0, ~5.3.0]
symfony: [^5.2]
Copy link
Member

Choose a reason for hiding this comment

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

With this provision, we are only testing 5.4 anyway, but as those versions (Sf 5.2 and 5.3) have not been supported for at least a year anyway, I think we can remove support for them clearly, as we did in Sylius 1.11

@GSadee GSadee merged commit ed55e6d into Sylius:1.6 Jan 5, 2023
@GSadee
Copy link
Member

GSadee commented Jan 5, 2023

Thank you, Kamil! 🥇

@TheMilek TheMilek deleted the drop-symfony-4.4 branch January 5, 2023 08:34
GSadee added a commit that referenced this pull request Jan 9, 2023
This PR was merged into the 1.6 branch.

Discussion
----------

Changes:

- Dropped `symfony/web-server-bundle` as is no longer supported by Symfony
- Bumped up the `phpstan/phpstan` to 1.8 version and `phpstan/phpstan-webmozart-assert` to 1.2
- Updated `.env.dist` files to make them usable with the available documentation

**Important!**
To potential merge after merging #735 and #737

Commits
-------

8657ed0 Support Sylius 1.12
8886e5e Update and fix PHPStan
ee2bd38 Remove redundant test
9387a1d fix .env files
e93b51d Fix Contract tests and ECS
0d7bd21 [PHPUnit] improve checking both swiftmailer and symfony mailer
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants