-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
[DependencyInjection] Add section about Service Closures #15731
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
Conversation
562d799
to
1dea8a0
Compare
…-DSL (HypeMC) This PR was merged into the 5.4 branch. Discussion ---------- [DependencyInjection] Add service_closure() to the PHP-DSL | Q | A | ------------- | --- | Branch? | 5.4 | Bug fix? | no | New feature? | yes | Deprecations? | no | Tickets | - | License | MIT | Doc PR | symfony/symfony-docs#15730 and symfony/symfony-docs#15731 Adds a `service_closure()` function to the PHP-DSL. Commits ------- f333fa0 [DI] Add service_closure() to the PHP-DSL
…-DSL (HypeMC) This PR was merged into the 5.4 branch. Discussion ---------- [DependencyInjection] Add service_closure() to the PHP-DSL | Q | A | ------------- | --- | Branch? | 5.4 | Bug fix? | no | New feature? | yes | Deprecations? | no | Tickets | - | License | MIT | Doc PR | symfony/symfony-docs#15730 and symfony/symfony-docs#15731 Adds a `service_closure()` function to the PHP-DSL. Commits ------- f333fa0e05 [DI] Add service_closure() to the PHP-DSL
1dea8a0
to
822bcec
Compare
822bcec
to
1e40081
Compare
Using Service Closures in Compiler Passes | ||
----------------------------------------- |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Using Service Closures in Compiler Passes | |
----------------------------------------- | |
Using Service Closures in Compiler Pass | |
--------------------------------------- |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe even use Service Closure (singular) in the headline?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What about "Using a Service Closures in a Compiler Pass"?
005216a
to
8ab82c2
Compare
service_closure()
function
ab6d0d0
to
b69cdaa
Compare
b69cdaa
to
1958c06
Compare
service_closure()
functionf391a2c
to
28f9b55
Compare
28f9b55
to
f2b2fdb
Compare
Thank you @HypeMC. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good job with this PR, I've left a small comment, sorry for the delay.
This PR was squashed before being merged into the 5.4 branch. Discussion ---------- Be consistent in code examples Follows #15731 (comment) cc `@HeahDude` Commits ------- 2426ce4 Be consistent in code examples
Documents service closures added in symfony/symfony#21770 and symfony/symfony#41176, and the
service_closure()
PHP-DSL function added in symfony/symfony#42625 .