-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
[Task]: Move specific functionalities into bundles to keep the core leaner #12094
Comments
Related #11105. |
@brusch will the bundles stay open source? |
@passioneight yes, of course. This task is just about restructuring and possibly also about priorities, but not about licensing 😊 |
Regarding |
@NiklasBr thanks for the hint, you're right, that should be deprecated, no reason to further support an unmaintained product. |
See #12431 |
Pimcore will still be a monorepo-right? Cause the issues you created read like the code will actually be moved to independent repos, instead of creating subtree splits of this monorepo. |
@jdreesen this is not yet finally decided, but currently our favorite is to separate them out into their own repos. There are multiple reasons for that, mostly organizational wise, e.g. this gives us a better control over permissions and responsibilities. Of course we also know about the drawbacks, but as we're currently only talking about additional features, which are not essential core features such as data objects, I think it would be still reasonable. |
IMO, the value of having it be a mono-repo heavily outweighs the drawbacks. Symfony proved the mono-repo approach works on a huge scale, I don't see why it would work for Pimcore. Managing many repos, keeping their settings, tags, GH actions etc in sync, making cross-cutting PRs, doing releases in sync will be a huge PITA otherwise. |
CoreShop also successfully uses the mono-repo approach and works flawless. Makes development a lot easier |
Strategy is as follows:
|
System InfoFile ExplorerPersonalizationGoogle MarketingWeb-to-printEcommerceAdmin UI ClassicNewsletterTasks:
Also consider #9357 for each functionality
The text was updated successfully, but these errors were encountered: