Skip to content

[Form] remove remaining deprecation layers #41743

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 19, 2021
Merged

Conversation

xabbuh
Copy link
Member

@xabbuh xabbuh commented Jun 18, 2021

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

@@ -72,9 +64,8 @@ public function createListFromLoader(ChoiceLoaderInterface $loader, callable $va
*
* @param array|callable $labelTranslationParameters The parameters used to translate the choice labels
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let's remove as many annotations as possible, and add the union types meanwhile!

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agreed, but I prefer to do that in a different PR.

@xabbuh xabbuh merged commit 0dc4d3b into symfony:6.0 Jun 19, 2021
@xabbuh xabbuh deleted the pr-41318 branch June 19, 2021 07:45
@fabpot fabpot mentioned this pull request Nov 5, 2021
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.

5 participants