Skip to content

Added a tip about the expanded YAML syntax #7376

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

Closed
wants to merge 2 commits into from

Conversation

javiereguiluz
Copy link
Member

This fixes #5107.


.. code-block:: yaml

services:
Copy link
Member

Choose a reason for hiding this comment

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

Should we add the filename (app/config/services.yml) as a comment?

@xabbuh
Copy link
Member

xabbuh commented Jan 17, 2017

👍

Status: Reviewed

@xabbuh
Copy link
Member

xabbuh commented Jan 31, 2017

Thank you @javiereguiluz.

xabbuh added a commit that referenced this pull request Jan 31, 2017
This PR was squashed before being merged into the 2.7 branch (closes #7376).

Discussion
----------

Added a tip about the expanded YAML syntax

This fixes #5107.

Commits
-------

a5ea7b1 Added a tip about the expanded YAML syntax
xabbuh added a commit that referenced this pull request Jan 31, 2017
@xabbuh xabbuh closed this Jan 31, 2017
xabbuh added a commit that referenced this pull request Feb 3, 2017
* 2.7:
  [#7427] fix line length
  Add apcu cache driver to doctrine config reference
  Change type of arguments min and max
  [#7383] minor rewording
  Added a note about not using the ClassLoader component
  More clear description of factory service creation
  [#7376] minor wording improvement
  Added a tip about the expanded YAML syntax
  Added a minor help note about Request::setTrustedProxies
  Fix Apache 2.4 UDS instructions
  See #7362 bundles/override > Rewrite translations block
xabbuh added a commit that referenced this pull request Feb 3, 2017
* 2.8:
  [#7427] fix line length
  Add apcu cache driver to doctrine config reference
  Change type of arguments min and max
  [#7383] minor rewording
  Added a note about not using the ClassLoader component
  Update guard_authentication.rst
  More clear description of factory service creation
  [#7376] minor wording improvement
  Added a tip about the expanded YAML syntax
  Added a minor help note about Request::setTrustedProxies
  Fix Apache 2.4 UDS instructions
  See #7362 bundles/override > Rewrite translations block
xabbuh added a commit that referenced this pull request Feb 3, 2017
* 3.1:
  [#7427] fix line length
  Add apcu cache driver to doctrine config reference
  Change type of arguments min and max
  [#7383] minor rewording
  Added a note about not using the ClassLoader component
  Update guard_authentication.rst
  More clear description of factory service creation
  [#7376] minor wording improvement
  Added a tip about the expanded YAML syntax
  Added a minor help note about Request::setTrustedProxies
  Fix Apache 2.4 UDS instructions
  remove unused options parameter from loadManifest
  See #7362 bundles/override > Rewrite translations block
xabbuh added a commit that referenced this pull request Feb 3, 2017
* 3.2:
  [#7427] fix line length
  Add apcu cache driver to doctrine config reference
  Change type of arguments min and max
  [#7383] minor rewording
  Added a note about not using the ClassLoader component
  Update guard_authentication.rst
  More clear description of factory service creation
  [#7376] minor wording improvement
  Added a tip about the expanded YAML syntax
  Added a minor help note about Request::setTrustedProxies
  Fix Apache 2.4 UDS instructions
  Fix doc for yaml parameters constants usage
  remove unused options parameter from loadManifest
  Tweaks based on feedback!
  Covering two missing adapters introduced in 3.2
  See #7362 bundles/override > Rewrite translations block
@javiereguiluz javiereguiluz deleted the fix_5107 branch May 24, 2018 16:05
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