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

Deprecated alias on DI Extension #1079

Merged
merged 1 commit into from
Nov 28, 2019

Conversation

maxhelias
Copy link
Contributor

Fix issue #1067

@maxhelias maxhelias changed the base branch from master to 1.12.x November 28, 2019 10:36
@maxhelias maxhelias force-pushed the deprecated-alias-bis branch from 20a36bf to 728f74d Compare November 28, 2019 10:38
@maxhelias
Copy link
Contributor Author

@stof Can you tell me if it's ok like that ?

@alcaeus alcaeus added the Bug label Nov 28, 2019
@alcaeus alcaeus added this to the 1.12.2 milestone Nov 28, 2019
DependencyInjection/DoctrineExtension.php Outdated Show resolved Hide resolved
DependencyInjection/DoctrineExtension.php Outdated Show resolved Hide resolved
@maxhelias maxhelias force-pushed the deprecated-alias-bis branch from 728f74d to 87b74f3 Compare November 28, 2019 12:34
@alcaeus alcaeus self-assigned this Nov 28, 2019
@alcaeus alcaeus merged commit 9ac0418 into doctrine:1.12.x Nov 28, 2019
@alcaeus
Copy link
Member

alcaeus commented Nov 28, 2019

Thanks @maxhelias!

@maxhelias maxhelias deleted the deprecated-alias-bis branch November 28, 2019 13:01
@stof
Copy link
Member

stof commented Nov 28, 2019

@alcaeus I suggest making a release for that, to avoid getting lots of bug reports for that from people using stable releases. What do you think ?

@alcaeus
Copy link
Member

alcaeus commented Nov 28, 2019

Done.

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

Successfully merging this pull request may close these issues.

3 participants