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

Support Symfony 6 and Sylius 1.12 #193

Merged
merged 5 commits into from
Dec 13, 2022
Merged

Conversation

TheMilek
Copy link
Member

No description provided.

@TheMilek TheMilek requested a review from a team as a code owner November 28, 2022 17:54
@TheMilek TheMilek force-pushed the support-symfony-6 branch 10 times, most recently from 3ab8b1e to bb95762 Compare November 29, 2022 07:58
@TheMilek TheMilek marked this pull request as draft November 29, 2022 22:26
@TheMilek TheMilek force-pushed the support-symfony-6 branch 3 times, most recently from ef086ad to 732d3a2 Compare December 5, 2022 12:38
@TheMilek TheMilek marked this pull request as ready for review December 5, 2022 13:02
@TheMilek TheMilek closed this Dec 5, 2022
@TheMilek TheMilek reopened this Dec 5, 2022
@TheMilek TheMilek closed this Dec 5, 2022
@TheMilek TheMilek reopened this Dec 5, 2022
@TheMilek TheMilek closed this Dec 5, 2022
@TheMilek TheMilek reopened this Dec 5, 2022
src/Form/Type/AdjustmentType.php Outdated Show resolved Hide resolved
src/Form/Type/AdjustmentType.php Outdated Show resolved Hide resolved
tests/Application/composer.lock Outdated Show resolved Hide resolved
@@ -20,11 +20,15 @@ jobs:
fail-fast: false
matrix:
php: ["8.0", "8.1"]
symfony: ["^4.4", "^5.4"]
Copy link
Member

Choose a reason for hiding this comment

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

It would be nice to drop support for Symfony 4.4 in a separate PR

@@ -29,6 +31,7 @@ public function buildForm(FormBuilderInterface $builder, array $options): void
]);

$builder->addEventListener(FormEvents::SUBMIT, function (FormEvent $event) use ($options): void {
/** @var Adjustment|null $adjustment */
Copy link
Member

Choose a reason for hiding this comment

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

Use interface here

@GSadee GSadee added the Maintenance Configurations, READMEs, releases, etc. label Dec 13, 2022
@GSadee GSadee merged commit a5e3c5f into Sylius:main Dec 13, 2022
@GSadee
Copy link
Member

GSadee commented Dec 13, 2022

Thank you, Kamil! 🎉

@TheMilek TheMilek deleted the support-symfony-6 branch December 13, 2022 08:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Maintenance Configurations, READMEs, releases, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants