Skip to content

[DependencyInjection] Add support for #[Autowire(lazy: true)] #18106

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
Mar 29, 2023

Conversation

MrYamous
Copy link
Contributor

Fixes #18095

I'm wondering if this argument should be documented here (where the attribute in mainly mentioned) or if it is fine to split considering the page about service lazy loading is relevant for this arg ?

@javiereguiluz javiereguiluz merged commit d8804cb into symfony:6.3 Mar 29, 2023
@javiereguiluz
Copy link
Member

Matthieu, I think it's enough to document this in the doc that you selected for this PR. Depending on the user feedback, we can add a link from the other article that you mentioned.

Thanks!

javiereguiluz added a commit that referenced this pull request Apr 5, 2023
…class-string)]` (MrYamous)

This PR was merged into the 6.3 branch.

Discussion
----------

[DependencyInjection] Add support for `#[Autowire(lazy: class-string)]`

In addition to issue #18095 / pr #18106 to explicitly document symfony/symfony#49836

Commits
-------

1fd4f07 complement lazy service autowiring
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.

3 participants