[RFC] Cleaner separation between Sylius and Sylius-Standard

[teohhanhui]

Q	A
Bug report?	no
Feature request?	no
BC Break report?	no
RFC?	yes
Sylius version	future major

Currently, many files exist in the Sylius repository that are only useful for the Sylius-Standard repository. This means the two projects are tightly coupled in a convoluted way. Issues like #9301 and #9304 cannot be properly fixed, because of the lack of a clean separation. Of course, this also makes it almost impossible to move to Symfony Flex (correct me if I'm wrong).

[pamil]

Apart of them being useful in Sylius-Standard, we also need them to test the whole application using Behat. I'm not sure if I understand Symfony Flex issue correctly, but it's mostly connected to single bundles, not the whole application, so this should be possible just by customising them.

[teohhanhui]

@pamil Continuing the discussion here.

We could start by getting rid of the shared Kernel, or if there's a need for some extended functionality, making sure that whatever is in it is not related to the "app" / distribution.

[teohhanhui]

Apart of them being useful in Sylius-Standard, we also need them to test the whole application using Behat.

The test app should have its own AppKernel. Yes, there will be some duplication, but it's necessary. :)

[teohhanhui]

Also, all the XXXBundle/Resources/app/config.yml and such should definitely be moved to Sylius-Standard.

Say, normally these issues would be fixed in Sylius-Standard, not Sylius:

• Disable deprecated "form mapping" feature in SonataCoreBundle #9330
• Fix exception controller config #9344
• (rolling list...)

[stale]

This issue has been automatically marked as stale because it has not had any recent activity. It will be closed in a week if no further activity occurs. Thank you for your contributions.

[pamil]

It's still a pain point from maintainer/contributor perspective.

[Zales0123]

Even though we will probably be thinking about what we should do with Sylius-Standard in the future, I believe this issue can be closed 🖖