Skip to content

[DependencyInjection] Update import.rst #20091

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
May 30, 2025
Merged

Conversation

GrinWay
Copy link
Contributor

@GrinWay GrinWay commented Aug 3, 2024

It's better to use imports statements only in the services.yaml file. The following explains that.

@GrinWay
Copy link
Contributor Author

GrinWay commented Aug 25, 2024

I just wanted to add important notes for the most recent version of this framework.
That's my first commit for the Symfony, I hope I made it in the right place... and it will not become lost.

@carsonbot carsonbot changed the title Update import.rst [DependencyInjection] Update import.rst May 30, 2025
@javiereguiluz javiereguiluz modified the milestones: 7.2, 6.4 May 30, 2025
It's better to use imports statements only in the services.yaml file.
The following explains that.
@javiereguiluz javiereguiluz changed the base branch from 7.1 to 6.4 May 30, 2025 07:09
@javiereguiluz javiereguiluz merged commit 3e0ea84 into symfony:6.4 May 30, 2025
2 of 3 checks passed
@javiereguiluz
Copy link
Member

Thanks Григорий! We merged this but made some rewords while merging to expand some explanations and to add examples for the three alternatives explained.

@GrinWay
Copy link
Contributor Author

GrinWay commented Jun 4, 2025

I'm so glad you merged it ❤️🐘

@GrinWay GrinWay deleted the patch-1 branch June 4, 2025 16:10
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