Skip to content

[DependencyInjection] Document abstract arguments #18633

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Aug 2, 2023
Merged

[DependencyInjection] Document abstract arguments #18633

merged 1 commit into from
Aug 2, 2023

Conversation

alamirault
Copy link
Contributor

@carsonbot carsonbot added this to the 6.4 milestone Jul 29, 2023
@alamirault alamirault changed the base branch from 6.4 to 5.4 July 29, 2023 20:13
@alamirault alamirault changed the title [DependencuInjection] Document abstract arguments [DependencyInjection] Document abstract arguments Jul 29, 2023
@alamirault alamirault mentioned this pull request Jul 29, 2023
Merged
javiereguiluz added a commit that referenced this pull request Jul 31, 2023
@javiereguiluz
minor #18634 Remove some unused use (alamirault)
b2f315d 
This PR was merged into the 5.4 branch.

Discussion
----------

Remove some unused use

Found when working on #18633

Commits
-------

546fc2f Remove some unused use
@javiereguiluz javiereguiluz modified the milestones: 6.4, 6.3 Aug 2, 2023
@javiereguiluz javiereguiluz merged commit 45873d6 into symfony:5.4 Aug 2, 2023
@javiereguiluz
Copy link
Member

Thank you Antoine. It's sad that this old feature wasn't documented yet ... but it's documented now thanks to you 🙌

@javiereguiluz javiereguiluz mentioned this pull request Aug 2, 2023
Closed
@alamirault alamirault deleted the feature/13064-abstract-arguments branch August 2, 2023 09:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
DependencyInjection Status: Reviewed
Projects
None yet
Milestone
6.3
Development

Successfully merging this pull request may close these issues.

[DI] added possibility to define services with abstract arguments
3 participants
@alamirault @javiereguiluz @carsonbot