Skip to content

[Routing][ObjectLoader] Remove forgotten deprecation after merge #34795

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
Dec 4, 2019

Conversation

fancyweb
Copy link
Contributor

@fancyweb fancyweb commented Dec 3, 2019

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

#34621 needs to be gone on 5.0. We want to throw here. I think it was forgotten when merging in 5.0.

@fancyweb fancyweb changed the title [Routing][ObjectLoader] Remove bad merge deprecation [Routing][ObjectLoader] Remove forgotten deprecation after merge Dec 3, 2019
@fancyweb fancyweb force-pushed the routing-remove-bad-merge-deprec branch from dce8ad3 to 4e60b9d Compare December 3, 2019 22:29
@nicolas-grekas nicolas-grekas added this to the 5.0 milestone Dec 3, 2019
@fabpot
Copy link
Member

fabpot commented Dec 4, 2019

Thank you @fancyweb.

fabpot added a commit that referenced this pull request Dec 4, 2019
… merge (fancyweb)

This PR was merged into the 5.0 branch.

Discussion
----------

[Routing][ObjectLoader] Remove forgotten deprecation after merge

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

#34621 needs to be gone on 5.0. We want to throw here. I think it was forgotten when merging in 5.0.

Commits
-------

4e60b9d [Routing][ObjectLoader] Remove forgotten deprecation after merge
@fabpot fabpot merged commit 4e60b9d into symfony:5.0 Dec 4, 2019
@fancyweb fancyweb deleted the routing-remove-bad-merge-deprec branch December 4, 2019 07:32
@fabpot fabpot mentioned this pull request Dec 19, 2019
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