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

ensure compatability with Symfony 4.4 #1016

Merged
merged 1 commit into from
Sep 16, 2019

Conversation

xabbuh
Copy link
Member

@xabbuh xabbuh commented Sep 13, 2019

No description provided.

@alcaeus
Copy link
Member

alcaeus commented Sep 13, 2019

Thanks @xabbuh! Do you want to take care of the other deprecations (adding missing return types) as well? The last deprecation will have to stay in 1.12.x, but this is one deprecation is ignored in the build itself.

If not, I can always fix this later today.

@xabbuh xabbuh changed the title implement getMetadataDriverClass() ensure compatability with Symfony 4.4 Sep 13, 2019
@alcaeus
Copy link
Member

alcaeus commented Sep 16, 2019

@xabbuh thanks for making the changes. I've fixed coding style issues, which should get the build stable. Am I correct to assume that the PHP 7.4 failures are going to be fixed with Symfony 4.4? If so, would it make sense to switch the PHP 7.4 build to use Symfony 4.4 as well, as we already do for one 7.3 build?

@xabbuh
Copy link
Member Author

xabbuh commented Sep 16, 2019

If the PHP 7.4 build failure is related to Symfony, it doesn't really matter which version we use as long as it is still supported.

@alcaeus
Copy link
Member

alcaeus commented Sep 16, 2019

I believe it may be related to twig, see https://travis-ci.org/doctrine/DoctrineBundle/jobs/585739666#L382:

18x: array_key_exists(): Using array_key_exists() on objects is deprecated. Use isset() or property_exists() instead
18x in ProfilerTest::testRender from Doctrine\Bundle\DoctrineBundle\Tests

Either way, this is good to go. Thanks!

@alcaeus alcaeus merged commit 4562ea0 into doctrine:1.12.x Sep 16, 2019
@xabbuh xabbuh deleted the symfony-4.4-compat branch September 17, 2019 06:25
@xabbuh
Copy link
Member Author

xabbuh commented Sep 17, 2019

see twigphp/Twig#3097 then (probably), so it will be fixed in the next Twig release

@alcaeus alcaeus self-assigned this Sep 18, 2019
@alcaeus alcaeus added this to the 1.12.0 milestone Sep 18, 2019
@alcaeus alcaeus added this to 1.12 in Roadmap Nov 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Roadmap
1.12 (bug fixes only)
Development

Successfully merging this pull request may close these issues.

None yet

3 participants