Skip to content

[ErrorHandler] Do not use ContextAwareNormalizerInterface anymore #50824

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
merged 1 commit into from
Jun 30, 2023

Conversation

lyrixx
Copy link
Member

@lyrixx lyrixx commented Jun 29, 2023

Q A
Branch? 6.4
Bug fix? no
New feature? no
Deprecations? no
Tickets
License MIT
Doc PR

It's a deprecated feature + #50736

@lyrixx lyrixx changed the base branch from 5.4 to 6.4 June 30, 2023 09:17
@nicolas-grekas nicolas-grekas modified the milestones: 5.4, 6.4 Jun 30, 2023
@nicolas-grekas
Copy link
Member

Thank you @lyrixx.

@nicolas-grekas nicolas-grekas merged commit 660b35a into symfony:6.4 Jun 30, 2023
@lyrixx lyrixx deleted the serializer-54 branch June 30, 2023 09:42
nicolas-grekas added a commit that referenced this pull request Jun 30, 2023
…(nicolas-grekas)

This PR was merged into the 6.4 branch.

Discussion
----------

[HttpKernel] Do not use ContainerAwareInterface anymore

| Q             | A
| ------------- | ---
| Branch?       | 6.4
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Tickets       | -
| License       | MIT
| Doc PR        | -

Similar to #50824, keeping this interface blocks compat of HK v6 with DI v7 for no good reasons. Let's drop ContainerAwareInterface from BundleInterface, it serves no purpose anyway.

Unlocks #50578

Commits
-------

640c72a [HttpKernel] Do not use ContainerAwareInterface anymore
nicolas-grekas added a commit that referenced this pull request Jul 4, 2023
This PR was merged into the 6.4 branch.

Discussion
----------

document BC breaking changes

| Q             | A
| ------------- | ---
| Branch?       | 6.4
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Tickets       |
| License       | MIT
| Doc PR        |

#50836 #50859 #50824

Commits
-------

9a22117 document BC breaking changes
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.

3 participants