Skip to content

[DependencyInjection] Add #[AutowireInline] attribute #19888

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
Closed
wants to merge 1 commit into from
Closed

[DependencyInjection] Add #[AutowireInline] attribute #19888

wants to merge 1 commit into from

Conversation

DaDeather
Copy link

Solves #19847

@carsonbot carsonbot added this to the 7.1 milestone May 14, 2024
@DaDeather DaDeather force-pushed the 19847-document-dependency-injection-new-inline-attribute branch from 8a74204 to 6edde8d Compare May 14, 2024 13:23
@OskarStark OskarStark changed the title [DependencyInjection] Add documentation for AutowireInline attribute [DependencyInjection] Add #[AutowireInline] attribute May 16, 2024
@DaDeather
Copy link
Author

Hey there just a quick question:
Is there still anything to do here for me 😀?

@nicolas-grekas nicolas-grekas mentioned this pull request May 12, 2025
Merged
javiereguiluz added a commit that referenced this pull request May 14, 2025
@javiereguiluz
minor #20958 [DependencyInjection] Tell about #[AutowireInline] (ni…
171f4f5 
…colas-grekas)

This PR was merged into the 7.2 branch.

Discussion
----------

[DependencyInjection] Tell about `#[AutowireInline]`

I feel like the doc proposed in #19888 is a bit too complex so I'd like to propose this simpler alternative.

Commits
-------

849acde [DI] Tell about #[AutowireInline]
@xabbuh
Copy link
Member

xabbuh commented May 14, 2025

Let's close here in favor of #20958 which is a bit simpler. @DaDeather Sorry it took us so long to react but thank you very much for kicking off these changes.

@xabbuh xabbuh closed this May 14, 2025
@DaDeather
Copy link
Author

Let's close here in favor of #20958 which is a bit simpler. @DaDeather Sorry it took us so long to react but thank you very much for kicking off these changes.

No worries and thanks for taking care of this 😀!

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: Needs Review
Projects
None yet
Milestone
7.1
Development

Successfully merging this pull request may close these issues.
3 participants
@DaDeather @xabbuh @carsonbot