Conversation
Remove legacy twig/twig version conflicts: - Remove ^3.0 conflict (added in 2019, already removed in Sylius 2.0+) - Remove ^1.0 conflict (Twig 1.x reached EOL years ago) Keep twig/twig:3.9.0 conflict as it's a specific buggy version documented in CONFLICTS.md. Since Twig 2.x reached EOL in December 2023, these conflicts blocked users of standalone bundles from upgrading to Twig 3.x. Closes #18700
ResourceBundle 1.12 moved several packages from require to require-dev. Bundle functional tests that use these packages now need explicit deps.
… require-dev ResourceBundle 1.12 moved willdurand/hateoas-bundle from require to require-dev. These bundles use BazingaHateoasBundle in their test kernels.
Version 2.17.0 adds AssetMapper support but breaks container compilation when AssetMapper is not being used.
in case certain xml keys are missing from extending config like properties, item opteration, collection operations or resources
Updates the `branch-alias` in all `composer.json` files from `1.13-dev` to `1.14-dev`. Affects all bundles and components (41 files).
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
| Q | A |-----------------|----- | Branch? | 1.13 <!-- see the comment below --> | Bug fix? | a bit | New feature? | no | BC breaks? | no | License | MIT <!-- - Bug fixes must be submitted against the 1.13 branch - Features and deprecations must be submitted against the 1.14 branch - Features, removing deprecations and BC breaks must be submitted against the 2.0 branch - Make sure that the correct base branch is set To be sure you are not breaking any Backward Compatibilities, check the documentation: https://docs.sylius.com/en/latest/book/organization/backward-compatibility-promise.html --> We need to adjust the code to make it consistent everywhere we use it hardcoded in our test app
| Q | A |-----------------|----- | Branch? | 1.14 | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Related tickets | fixes #X, partially #Y, mentioned in #Z | License | MIT Fix the broken romanian plural translation. There are multiple other keys in the translation.
| Q | A |-----------------|----- | Branch? | 1.14 <!-- see the comment below --> | Bug fix? | yes | New feature? | no | BC breaks? | no | License | MIT <!-- - Bug fixes must be submitted against the 1.14 or 2.1 branch - Features and deprecations must be submitted against the 2.2 branch - Make sure that the correct base branch is set To be sure you are not breaking any Backward Compatibilities, check the documentation: https://docs.sylius.com/en/latest/book/organization/backward-compatibility-promise.html --> continuation of #16146
…d add upgrade file to inform about recommendations
Remove legacy `twig/twig` version conflicts that blocked users of standalone bundles from upgrading to Twig 3.x: - Remove `^3.0` conflict added in 2019, already removed in Sylius 2.0+ - Remove `^1.0` conflict (Twig 1.x reached EOL years ago) - Keep `3.9.0` conflict as it's a specific buggy version documented in `CONFLICTS.md` Since Twig 2.x reached EOL in December 2023, these conflicts were blocking users who use Sylius bundles standalone from upgrading. Also adds `friendsofsymfony/rest-bundle` and `jms/serializer-bundle` to UserBundle `require-dev` - they're used in functional tests but were previously installed as transitive dependencies of `sylius/resource-bundle` (moved from `require` to `require-dev` in ResourceBundle 1.13). Additionally adds conflict for `liip/imagine-bundle: 2.17.0` - this version was released on 2026-01-05 and breaks container compilation when AssetMapper is not used (unrelated to twig changes). Affected bundles: AddressingBundle, CustomerBundle, CurrencyBundle, InventoryBundle, LocaleBundle, MoneyBundle, OrderBundle, PaymentBundle, PromotionBundle, ReviewBundle, ShippingBundle, TaxationBundle, TaxonomyBundle, UiBundle, UserBundle Closes #18700
❌ Preview Environment deleted from BunnyshellAvailable commands:
|
7bf6291 to
ab6f423
Compare
TheMilek
approved these changes
Jan 19, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR has been generated automatically.
For more details see upmerge_pr.yaml.
Remember! The upmerge should always be merged with using
Merge pull requestbutton.In case of conflicts, please resolve them manually with usign the following commands:
If you use other name for the upstream remote, please replace
upstreamwith the name of your remote pointing to theSylius/Syliusrepository.Once the conflicts are resolved, please run
git merge --continueand push the changes to this PR.