Skip to content

[Doctrine][Repository] deprecated replacement #454

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

Merged
merged 1 commit into from
Aug 12, 2019

Conversation

maxhelias
Copy link
Contributor

Application of symfony/symfony#32817

@weaverryan
Copy link
Member

And because the ManagerRegistry has "always" (since the days of autowiring) been a working alias to the doctrine service, this should work everywhere immediately.

@weaverryan
Copy link
Member

Thank you @maxhelias!

@weaverryan weaverryan merged commit 74f5135 into symfony:master Aug 12, 2019
weaverryan added a commit that referenced this pull request Aug 12, 2019
This PR was merged into the 1.0-dev branch.

Discussion
----------

[Doctrine][Repository] deprecated replacement

Application of symfony/symfony#32817

Commits
-------

74f5135 [Doctrine][Repository] deprecated replacement
weaverryan added a commit to weaverryan/maker-bundle that referenced this pull request Aug 13, 2019
weaverryan added a commit to weaverryan/maker-bundle that referenced this pull request Aug 13, 2019
weaverryan added a commit that referenced this pull request Aug 13, 2019
This PR was squashed before being merged into the 1.0-dev branch (closes #455).

Discussion
----------

Minimizing test matrix + fixing tests

Trying to strike a balance between having a very robust test suite, and one that takes too long to execute & blocks jobs under the `symfony/` organization.

Commits
-------

6d75f13 Adding support for new ErrorHandler DebugClassLoader
cd4ec4e changing from RegistryInterface to ManagerRegistry - #454
e622635 adding mysql service
3ebce10 Slightly minimizing our test matrix to try to reduce burden on Travis
@maxhelias maxhelias deleted the deprecated-repository branch August 16, 2019 15:11
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