Skip to content

build: add conflict with twig/twig 3.29.0 - #383

Merged
loic425 merged 2 commits into
Sylius:mainfrom
stlgaits:twig-conflict
Sep 23, 2026
Merged

loic425 merged 2 commits into
Sylius:mainfrom
stlgaits:twig-conflict

Conversation

@stlgaits

@stlgaits stlgaits commented Sep 21, 2026 •

Copy link
Copy Markdown
Contributor

@stlgaits stlgaits self-assigned this Sep 21, 2026
@stlgaits stlgaits added the bug Something isn't working label Sep 21, 2026
@stlgaits
stlgaits marked this pull request as ready for review September 21, 2026 09:43
@stlgaits stlgaits changed the title build: add conflict with twig/twig 3.29 build: add conflict with twig/twig 3.29.* Sep 21, 2026

@Prometee Prometee left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@stlgaits This is a sensible safety net, but I’d suggest tightening the constraint to the exact broken range if upstream has confirmed that only the affected Twig 3.29.x line is impacted. As written, twig/twig: 3.29.* blocks every 3.29 release, even for projects that already include the fixed Symfony/Sylius patches. A narrower range (or at least a clear comment with the upstream fix references) will make the workaround easier to remove once the fixed packages are released.

@TheMilek TheMilek changed the title build: add conflict with twig/twig 3.29.* build: add conflict with twig/twig 3.29.0 Sep 23, 2026
@loic425
loic425 merged commit ac3f777 into Sylius:main Sep 23, 2026
6 checks passed
@loic425

loic425 commented Sep 23, 2026

Copy link
Copy Markdown
Member

Thx @stlgaits and @TheMilek 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants