Skip to content
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

Bumps twig to 2.12/3.0 for PHP 7.4 compatibility #21

Merged
merged 1 commit into from Dec 11, 2019

Conversation

Intrepidity
Copy link
Contributor

@Intrepidity Intrepidity commented Dec 3, 2019

2.12 fixes an issue where array_key_exists is used on objects, which generates an E_DEPRECATED level error in PHP 7.4

@Intrepidity
Copy link
Contributor Author

@victorwelling Would you have time to take a look at this one?

@victorwelling
Copy link
Member

@Intrepidity Apologies, it appears I don't get notifications for new PR's. Would you be OK with including v3.0 of Twig as well?

@Intrepidity
Copy link
Contributor Author

I didn't test it with 3.0, not sure how many internals changed that would make it not work with shoot. I'll take some time to test it and get back to you :)

@victorwelling
Copy link
Member

I've been testing Shoot with v3.0 since it's been in development, so it should be fine 😬

@Intrepidity
Copy link
Contributor Author

Intrepidity commented Dec 10, 2019

I think we should then stick with ^2.12.0 || ^3.0.0, any future 2.x version would be fine as well.

@Intrepidity Intrepidity changed the title Bumps twig to 2.12 for PHP 7.4 compatibility Bumps twig to 2.12/3.0 for PHP 7.4 compatibility Dec 10, 2019
@Intrepidity
Copy link
Contributor Author

Changed to 2.12 and 3.0

@victorwelling victorwelling merged commit 1521944 into shootphp:master Dec 11, 2019
@victorwelling
Copy link
Member

@Intrepidity: Shoot v3.2 is now available. Happy holidays!

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

Successfully merging this pull request may close these issues.

None yet

2 participants