symfony/symfony-docsPublic

NotificationsYou must be signed in to change notification settings
Fork5.3k
Star2.3k

Documented short DI tag syntax#7769

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

Jump to bottom

Merged

xabbuh merged 1 commit intosymfony:masterfromderrabus:3.3-short-di-tags-in-yaml

Apr 8, 2017

Merged

Documented short DI tag syntax#7769

xabbuh merged 1 commit intosymfony:masterfromderrabus:3.3-short-di-tags-in-yaml

Apr 8, 2017

Conversation

Copy link

Member

derrabus commentedApr 8, 2017

This PRfixes#7248 and documents the short DI tag syntax for YAML files.

Documented short DI tag syntax.

f5b1f23

carsonbot added the Status: Needs Review label

Apr 8, 2017

Copy link

MemberAuthor

derrabus commentedApr 8, 2017

ping@xabbuh

wouterj approved these changes

Apr 8, 2017

View reviewed changes

wouterj added Status: Reviewed and removed Status: Needs Review labels

Apr 8, 2017

xabbuh added this to the3.3 milestone

Apr 8, 2017

Copy link

Member

xabbuh commentedApr 8, 2017

Thank you Alexander.

xabbuh merged commitf5b1f23 intosymfony:master

Apr 8, 2017

xabbuh added a commit that referenced this pull request

Apr 8, 2017

feature#7769Documented short DI tag syntax (derrabus)

7849331

This PR was merged into the master branch.Discussion----------Documented short DI tag syntaxThis PRfixes#7248 and documents the short DI tag syntax for YAML files.Commits-------f5b1f23 Documented short DI tag syntax.

xabbuh added a commit that referenced this pull request

Apr 8, 2017

[#7769] tweak line break positions

4251e0e

xabbuh added a commit that referenced this pull request

Apr 8, 2017

[#7769] add missing versionadded directive

e47090e

Copy link

Member

xabbuh commentedApr 8, 2017

After merging I realised that we should add aversionadded directive to not confuse users of older Symfony versions. So I added one ine47090e.

derrabus deleted the 3.3-short-di-tags-in-yaml branch

April 8, 2017 11:12

GuilhemN mentioned this pull request

Apr 30, 2017

Use the short tag syntax#7863

Merged

javiereguiluz added a commit that referenced this pull request

May 2, 2017

minor#7863Use the short tag syntax (GuilhemN)

ef36cc7

This PR was merged into the master branch.Discussion----------Use the short tag syntaxContinuation of#7769: use the short tag syntax where possible.Commits-------5b596b9 Use the short tag syntax