Skip to content

[Container] Small typos #19546

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
Feb 12, 2024
Merged

Conversation

chadyred
Copy link
Contributor

@chadyred chadyred commented Feb 11, 2024

@carsonbot carsonbot added this to the 7.1 milestone Feb 11, 2024
@chadyred chadyred changed the title [Container] Bad tag name [Container] Small typos Feb 11, 2024
@@ -1257,7 +1257,7 @@ unique string as the key of each service config:
event_subscribers:
namespace: App\Domain\
resource: '../src/Domain/*/EventSubscriber'
tags: [event_subscriber]
tags: [kernel.event_subscriber]
Copy link
Member

Choose a reason for hiding this comment

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

I think that the example deliberately used event_subscriber as a tag (matching the custom command_handler tag used above).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, I checkout this change, thanks

@xabbuh xabbuh changed the base branch from 7.1 to 5.4 February 12, 2024 15:02
@xabbuh xabbuh force-pushed the doc/typo-service-container branch from 2abf844 to 9b1c4b9 Compare February 12, 2024 15:02
@xabbuh
Copy link
Member

xabbuh commented Feb 12, 2024

Good catch, thanks @chadyred.

@xabbuh xabbuh merged commit 7b9d38f into symfony:5.4 Feb 12, 2024
@chadyred chadyred deleted the doc/typo-service-container branch February 12, 2024 15:20
@chadyred
Copy link
Contributor Author

You are welcome @xabbuh, thanks for your time and if you have a little bit more time, my old PR in the same spirit 🐴 #19395 :D

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.

4 participants