Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[DependencyInjection] Attribute documentation #17563

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
9 tasks done
kbond opened this issue Dec 14, 2022 · 4 comments
Closed
9 tasks done

[DependencyInjection] Attribute documentation #17563

kbond opened this issue Dec 14, 2022 · 4 comments
Labels
DependencyInjection Keep open

Comments

@kbond
Copy link
Member

kbond commented Dec 14, 2022 •
edited by OskarStark
Loading

I'm opening this as a meta issue to track the documentation status of the DI attributes. I ran into some confusion about AsTaggedItem in symfony/symfony#48532 which prompted me to look for docs which are missing.
@OskarStark OskarStark changed the title [Dependency Injection] Attribute documentation [DependencyInjection] Attribute documentation Dec 14, 2022
@alexandre-daubois
Copy link
Member

alexandre-daubois commented Jan 8, 2023 •
edited
Loading

Hey! Little update on this one.
TaggedIterator: #17652 (merged)
TaggedLocator: #17662 (merged)
AsTaggedItem: #17694
Autoconfigure + AutoconfigureTag : #17695
Target: #17696

This was referenced Jan 8, 2023
Merged
Merged
Merged
@kbond
Copy link
Member Author

kbond commented Jan 9, 2023

Awesome work @alexandre-daubois! I've updated the description.

OskarStark added a commit that referenced this issue Jan 10, 2023
@OskarStark
feature #17696 [DependencyInjection] Add #[Target] documentation (a…
29ad979 
…lexandre-daubois)

This PR was merged into the 5.4 branch.

Discussion
----------

[DependencyInjection] Add `#[Target]` documentation

Closes #15261

Part of #17563

Commits
-------

ce8739e [DependencyInjection] Add #[Target] documentation
OskarStark added a commit that referenced this issue Jan 10, 2023
@OskarStark
feature #17694 [DependencyInjection] Add #[AsTaggedItem] documentat…
9e42c2b 
…ion (alexandre-daubois)

This PR was merged into the 5.4 branch.

Discussion
----------

[DependencyInjection] Add `#[AsTaggedItem]` documentation

Part of #17563

Commits
-------

49e2405 [DependencyInjection] Add #[AsTaggedItem] documentation
@OskarStark
Copy link
Contributor

I updated the checkboxes, only one left 👍

OskarStark added a commit that referenced this issue Jan 10, 2023
@OskarStark
feature #17695 [DependencyInjection] Add #[Autoconfigure] and `#[Au…
cd9e68a 
…toconfigureTag]` documentation (alexandre-daubois)

This PR was merged into the 5.4 branch.

Discussion
----------

[DependencyInjection] Add `#[Autoconfigure]` and `#[AutoconfigureTag]` documentation

Closes #14990

Part of #17563

Commits
-------

ce16301 [DependencyInjection] Add #[Autoconfigure] and #[AutoconfigureTag] documentation
@OskarStark
Copy link
Contributor

All done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
DependencyInjection Keep open
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@kbond @wouterj @OskarStark @alexandre-daubois