NotificationsYou must be signed in to change notification settings
Fork9.7k
Star30.8k

[DI] Add missing deprecation in ContainerBuilder::addCompilerPass#20410

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Merged

fabpot merged 1 commit intosymfony:masterfromnicolas-grekas:deprec-no-prio

Nov 4, 2016

Merged

[DI] Add missing deprecation in ContainerBuilder::addCompilerPass#20410

fabpot merged 1 commit intosymfony:masterfromnicolas-grekas:deprec-no-prio

Nov 4, 2016

Conversation

Copy link

Member

nicolas-grekas commentedNov 4, 2016

Q	A
Branch?	3.2
Bug fix?	no
New feature?	yes
BC breaks?	no
Deprecations?	no
Tests pass?	yes
Fixed tickets	#18022
License	MIT
Doc PR	-

ping @Ener-Getick

[DI] Add missing deprecation in ContainerBuilder::addCompilerPass

c0e880e

carsonbot added Status: Needs Review Feature labels

Nov 4, 2016

Copy link

Member

fabpot commentedNov 4, 2016

Good catch, thanks@nicolas-grekas.

fabpot merged commitc0e880e intosymfony:master

Nov 4, 2016

fabpot added a commit that referenced this pull request

Nov 4, 2016

minor#20410[DI] Add missing deprecation in ContainerBuilder::addCom…

79259aa

…pilerPass (nicolas-grekas)This PR was merged into the 3.2-dev branch.Discussion----------[DI] Add missing deprecation in ContainerBuilder::addCompilerPass| Q             | A| ------------- | ---| Branch?       | 3.2| Bug fix?      | no| New feature?  | yes| BC breaks?    | no| Deprecations? | no| Tests pass?   | yes| Fixed tickets |#18022| License       | MIT| Doc PR        | -ping @Ener-GetickCommits-------c0e880e [DI] Add missing deprecation in ContainerBuilder::addCompilerPass