Skip to content

Conversation

yceruto
Copy link
Member

@yceruto yceruto commented Jan 13, 2018

@javiereguiluz javiereguiluz added the Waiting Code Merge Docs for features pending to be merged label Jan 13, 2018
javiereguiluz added a commit that referenced this pull request Jan 13, 2018
This PR was merged into the 2.7 branch.

Discussion
----------

[TwigBundle] Fix default value of strict_variables

See https://github.com/symfony/symfony/blob/d78a98d793c2846d6decbff652d63f981e4f32f4/src/Symfony/Bundle/TwigBundle/DependencyInjection/Configuration.php#L177

However, I'm proposing revert it since 4.1 and fix it inside the configuration section, see #9049

Commits
-------

3888217 Fix default value of strict_variables
@xabbuh xabbuh added this to the 4.1 milestone Jan 15, 2018
@yceruto
Copy link
Member Author

yceruto commented Jan 21, 2018

Hey, the PR's (symfony/symfony#25780) goal has changed (see symfony/symfony#25780 (comment)), we can't do this in 4.1 (BC break), so closing by now.

@yceruto yceruto closed this Jan 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Needs Review Waiting Code Merge Docs for features pending to be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants