-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
Short array syntax #10875
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
Short array syntax #10875
Conversation
@ricknox when you think this is ready to merge, please tell us. Thanks! |
@javiereguiluz I'm trying to finish this up tomorrow, hopefully. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for making docs easier to read :)
Co-Authored-By: ricknox <ricardo@43.nl>
Co-Authored-By: ricknox <ricardo@43.nl>
Thank you @jmsche for your improvements. |
I'm going to merge this now. If we find other missing changes, we can fix them later. Thanks. |
Thanks Ricardo. |
This PR was merged into the 3.4 branch. Discussion ---------- Short array syntax Now that symfony/symfony/pull/29811 is done we need to reflect these changes in the documentation. Commits ------- eac8e90 Apply suggestions/improvements from code review. 535d3a9 Apply suggestions/improvements from code review. a4440f9 chore: (2/2) Replace long with short syntax. 96f68fe chore: (1/2) Replace long with short syntax. 10e3a9e fix: Swap long with short syntax.
This has now been merged up to 4.1, 4.2 and master branches. I fixed a lot of |
Now that symfony/symfony/pull/29811 is done we need to reflect these changes in the documentation.