8000 [DependencyInjection] Add argument type `closure` to help passing closu… · Issue #16667 · symfony/symfony-docs · GitHub
[go: up one dir, main page]

Skip to content

[DependencyInjection] Add argument type closure to help passing closu… #16667

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

Closed
fabpot opened this issue Mar 30, 2022 · 2 comments · Fixed by #17792
Closed

[DependencyInjection] Add argument type closure to help passing closu… #16667

fabpot opened this issue Mar 30, 2022 · 2 comments · Fixed by #17792
Labels
DependencyInjection hasPR A Pull Request has already been submitted for this issue.
Milestone

Comments

@fabpot
Copy link
Member
fabpot commented Mar 30, 2022
Q A
Feature PR symfony/symfony#45878
PR author(s) @nicolas-grekas
Merged in 6.1
@xabbuh xabbuh added this to the 6.1 milestone Apr 1, 2022
@wouterj
Copy link
Member
wouterj commented May 1, 2022

@nicolas-grekas do you have time to write a few words about closure argument type in the service container documentation before the 6.1 release? :)

I think either a subsection on https://symfony.com/doc/current/service_container.html or a new sub article are the best locations (given the current structure of service container).

@wouterj wouterj added the actionable Clear and specific issues ready for anyone to take them. label May 1, 2022
Copy link
Member

I'm on it 🙂

@wouterj wouterj removed the actionable Clear and specific issues ready for anyone to take them. label Jan 27, 2023
@xabbuh xabbuh added the hasPR A Pull Request has already been submitted for this issue. label Jan 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DependencyInjection hasPR A Pull Request has already been submitted for this issue.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants
0