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

Symfony 2.7 deprecated notes fixed #86

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

e-moe
Copy link
Contributor

@e-moe e-moe commented Jun 2, 2015

@thewilkybarkid , pls check this changes due to latest Symfony 2.7 LTS release.

@xabbuh
Copy link
Contributor

xabbuh commented Jun 2, 2015

@e-moe Your change means that the bundle will not work with versions of the Symfony DependencyInjection component prior to 2.6. You can have a look at, for example, doctrine/DoctrineBundle#381 to see a BC approach to solve this issue.

@e-moe
Copy link
Contributor Author

e-moe commented Jun 2, 2015

thanks, I'll check and push update

@e-moe
Copy link
Contributor Author

e-moe commented Jun 3, 2015

@xabbuh , @thewilkybarkid done (DependencyInjection component BC fixed)

@xabbuh
Copy link
Contributor

xabbuh commented Jun 4, 2015

👍

@jiboule
Copy link

jiboule commented Jun 28, 2016

Is this going to be merged someday ?

@rvgate
Copy link

rvgate commented Jun 28, 2016

@jiboule dont expect anything to happen with this bundle anymore... check #90

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.

4 participants