Skip to content

Bump minimum twig version to 3.21#3656

Merged
bobvandevijver merged 1 commit into6.0from
bump-twig
Dec 15, 2025
Merged

Bump minimum twig version to 3.21#3656
bobvandevijver merged 1 commit into6.0from
bump-twig

Conversation

@bobvandevijver
Copy link
Member

With 3bbb001 we updated the code to no longer use deprecated calls, but the update requires at least twig 3.21.

Copilot AI review requested due to automatic review settings December 15, 2025 15:09
@bobvandevijver bobvandevijver merged commit b0c734f into 6.0 Dec 15, 2025
35 checks passed
@bobvandevijver bobvandevijver deleted the bump-twig branch December 15, 2025 15:22
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the minimum required version of Twig from 3.3 to 3.21 to align with a previous code change that removed deprecated Twig API calls.

  • Updated Twig version constraint in composer.json
  • Added changelog entry documenting the version requirement change

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
composer.json Updated twig/twig version constraint from ^3.3 to ^3.21
CHANGELOG.md Added entry documenting the new Twig minimum version requirement

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

- `doctrine/doctrine-fixtures-bundle` now requires `^4.3`.
- Support for `symfony/flex` v1 was dropped
- Support for `symfony/flex` v1 was dropped.
- `twig/twig` now requires `^3.21`.
Copy link

Copilot AI Dec 15, 2025

Choose a reason for hiding this comment

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

Corrected verb form: 'required' should be 'requires' to match the tense used in other entries in the list.

Suggested change
- `twig/twig` now requires `^3.21`.
- `twig/twig` now requires `^3.21`

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants