Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Add section about anonymous services in YAML #10041

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
Jul 11, 2018

Conversation

codedmonkey
Copy link
Contributor

Fixes #7694.

This is a continuation of #9909 which added some information about anonymous services in XML to the docs, next will be a commit that describes configuring anonymous services in PHP for the master branch. For further information about the issue, see #5854.

@javiereguiluz
Copy link
Member

@codedmonkey thanks for this contribution ... and thanks for your patience to create all these pull requests related to this feature

@javiereguiluz javiereguiluz merged commit 8a83664 into symfony:3.4 Jul 11, 2018
javiereguiluz added a commit that referenced this pull request Jul 11, 2018
…aan)

This PR was merged into the 3.4 branch.

Discussion
----------

Add section about anonymous services in YAML

Fixes #7694.

This is a continuation of #9909 which added some information about anonymous services in XML to the docs, next will be a commit that describes configuring anonymous services in PHP for the `master` branch. For further information about the issue, see #5854.

Commits
-------

8a83664 Add section about anonymous services in YAML
arguments:
- !service
class: AppBundle\AnonymousBar
autowire: true
Copy link
Member

Choose a reason for hiding this comment

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

Is this needed given that the default for this file is true too?

javiereguiluz added a commit that referenced this pull request Jul 12, 2018
This PR was merged into the 3.4 branch.

Discussion
----------

Remove redundant example configuration value

Fixes a small issue that was pointed out by @xabbuh in #10041 (review)

Commits
-------

594cf2c Remove redundant example configuration value
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
Morty Proxy This is a proxified and sanitized view of the page, visit original site.