Skip to content

[FrameworkBundle] add forward compatibility layer for FrameworkBundle #22774

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

Conversation

hhamon
Copy link
Contributor

@hhamon hhamon commented May 19, 2017

Q A
Branch? 3.4
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets ~
License MIT
Doc PR ~

@hhamon
Copy link
Contributor Author

hhamon commented May 19, 2017

This PR also depends on #22773.

…kBundle in version 3.4 with DependencyInjection component in version 4.0
@hhamon hhamon force-pushed the framework-bundle-forward-compat-di branch from 5e4ad56 to 9f14d06 Compare May 19, 2017 10:03
@stof
Copy link
Member

stof commented May 19, 2017

@hhamon technically, the dependency is the opposite one. The removal is the one depending on this one

@nicolas-grekas
Copy link
Member

I guess a rebase is need to make tests green

@nicolas-grekas
Copy link
Member

Thank you @hhamon.

@nicolas-grekas nicolas-grekas merged commit 9f14d06 into symfony:3.4 May 21, 2017
nicolas-grekas added a commit that referenced this pull request May 21, 2017
…eworkBundle (hhamon)

This PR was merged into the 3.4 branch.

Discussion
----------

[FrameworkBundle] add forward compatibility layer for FrameworkBundle

| Q             | A
| ------------- | ---
| Branch?       | 3.4
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | ~
| License       | MIT
| Doc PR        | ~

Commits
-------

9f14d06 [FrameworkBundle] add forward compatibility layer when using FrameworkBundle in version 3.4 with DependencyInjection component in version 4.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants