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
[DependencyInjection][3.0] Add initialized to container interface #15347
[DependencyInjection][3.0] Add initialized to container interface #15347
Conversation
znerol
commented
Jul 23, 2015
Q | A |
---|---|
Bug fix? | no |
New feature? | no |
BC breaks? | yes |
Deprecations? | no |
Tests pass? | yes |
Fixed tickets | #15346 |
License | MIT |
Doc PR | - |
This is a BC break, isn't it? As people typehinting for the new interface will have to update their code for Symfony 3.0 |
Yes, it is a BC, everyone who hast implemented |
Why not make a separate optional interface? |
@hhamon thats, what it was before...or i don't understand what you mean |
@fabpot thanks for the link, I didn't know there was a discussion about it. |
4e2f183
to
48a277d
Compare
👍 |
48a277d
to
bfddac3
Compare
Last commit resolves merge conflicts due to the recent introduction of |
👍 |
Thank you @znerol. |
…r interface (znerol) This PR was merged into the 3.0-dev branch. Discussion ---------- [DependencyInjection][3.0] Add initialized to container interface | Q | A | ------------- | --- | Bug fix? | no | New feature? | no | BC breaks? | yes | Deprecations? | no | Tests pass? | yes | Fixed tickets | #15346 | License | MIT | Doc PR | - Commits ------- bfddac3 [DependencyInjection] Add initialized() to the ContainerInterface
@fabpot should ResettableContainerInterface stay a separate interface in 3.0 or no ? |
@stof I think it should not. |