Skip to content
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

Split advanced container configuration article #5284

Merged
merged 2 commits into from
Jun 11, 2015

Conversation

wouterj
Copy link
Member

@wouterj wouterj commented May 23, 2015

Q A
Doc fixes? yes
New docs? yes
Applies to all
Fixed tickets #4228 (partially, 2 and 3)

@wouterj wouterj changed the title [WIP] Split advanced container configuration article Split advanced container configuration article May 23, 2015

The ``kernel`` service is called a synthetic service. This service has to be
configured in the container, so the container knows the service does exists
during compilation (otherwise, services dependening on this ``kernel`` service
Copy link
Member

Choose a reason for hiding this comment

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

typo: depending

@xabbuh
Copy link
Member

xabbuh commented Jun 9, 2015

👍

@weaverryan
Copy link
Member

Very cool. Let's face-lift the "definitions" entry too if you see fit (per Stof's comment)

@weaverryan weaverryan merged commit e3c2603 into symfony:2.3 Jun 11, 2015
weaverryan added a commit that referenced this pull request Jun 11, 2015
This PR was merged into the 2.3 branch.

Discussion
----------

Split advanced container configuration article

| Q | A
| --- | ---
| Doc fixes? | yes
| New docs? | yes
| Applies to | all
| Fixed tickets | #4228 (partially, 2 and 3)

Commits
-------

e3c2603 [#4228] Move synthetic services to its own recipe
26c7813 [#4228] Moved requiring files to definitions
@wouterj wouterj deleted the issue_4228 branch June 19, 2015 22:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants