Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

[DependencyInjection] [LazyServices] Add#[Autoconfigure] attribute mention#17784

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Conversation

@alexandre-daubois
Copy link
Member

While watching@nicolas-grekas' SymfonyCon talk about lazy objects to give a try to#16970, he told that the documentation about#[Autoconfigure(lazy: ...)] was missing. So here it is 😄

OskarStark reacted with laugh emojiOskarStark reacted with rocket emoji
@carsonbotcarsonbot added this to the5.4 milestoneJan 21, 2023
Copy link
Member

@nicolas-grekasnicolas-grekas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

LGTM.
Can you also please remove all occurrences of ProxyManager from the doc v6.2? It's not needed anymore.

alexandre-daubois reacted with hooray emoji
@alexandre-daubois
Copy link
MemberAuthor

alexandre-daubois commentedJan 21, 2023•
edited
Loading

Of course! I'll take care of this within the PR about the 2 traits 👍

@carsonbotcarsonbot changed the title[LazyServices] Add#[Autoconfigure] attribute mention[DependencyInjection] [LazyServices] Add#[Autoconfigure] attribute mentionJan 23, 2023
@javiereguiluzjaviereguiluz merged commit0aa2d60 intosymfony:5.4Jan 23, 2023
@javiereguiluz
Copy link
Member

Thanks!

alexandre-daubois reacted with hooray emoji

@alexandre-dauboisalexandre-daubois deleted the autoconfigure-attribute-laziness branchJanuary 23, 2023 11:30
javiereguiluz added a commit that referenced this pull requestJun 8, 2023
…documentation (alexandre-daubois)This PR was merged into the 6.2 branch.Discussion----------[VarExporter] Add `LazyGhostTrait` and `LazyProxyTrait` documentationFriendly ping `@nicolas`-grekas for your review on this technical one. I based myself a lot on the README of VarExporter that was filled-in (thanks for this, that was some priceless help here!) and your SymfonyCon talk.About your comment#17784 (review), it seems that occurrences of ProxyManager have already been removed for 6.2 and above. 🙂Fixes#16970Fixes#17249Commits-------ae0f85f [VarExporter] Add `LazyGhostTrait` and `LazyProxyTrait` documentation
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@nicolas-grekasnicolas-grekasnicolas-grekas left review comments

Assignees

No one assigned

Projects

None yet

Milestone

5.4

Development

Successfully merging this pull request may close these issues.

4 participants

@alexandre-daubois@javiereguiluz@nicolas-grekas@carsonbot
[8]ページ先頭
©2009-2025 Movatter.jp