Skip to content

Completed framework config #5298

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 3 commits into from
May 24, 2015
Merged

Completed framework config #5298

merged 3 commits into from
May 24, 2015

Conversation

wouterj
Copy link
Member

@wouterj wouterj commented May 23, 2015

This PR only changes and adds new configuration, without moving all things around. This will things a lot easier to diff and see where things have gone wrong. I'll thouroughly review this PR and merge then.

Replaces #4736

Q A
Doc fix? yes
New docs? yes
Applies to 2.3+
Fixed tickets #4439, #3586

@wouterj wouterj merged commit 54e8b0a into symfony:2.3 May 24, 2015
wouterj added a commit that referenced this pull request May 24, 2015
This PR was merged into the 2.3 branch.

Discussion
----------

Completed framework config

This PR only changes and adds new configuration, without moving all things around. This will things a lot easier to diff and see where things have gone wrong. I'll thouroughly review this PR and merge then.

Replaces #4736

| Q | A
| --- | ---
| Doc fix? | yes
| New docs? | yes
| Applies to | 2.3+
| Fixed tickets | #4439, #3586

Commits
-------

54e8b0a Let docbot review reference
f223643 Apply some of Ryan's suggestions
7e45e8f Completed framework config reference
@wouterj wouterj deleted the config_ref_changes branch May 24, 2015 21:15
@wouterj
Copy link
Member Author

wouterj commented May 24, 2015

So this one is finally merged. A huge thanks to everyone who took some minutes/hours to review the previous PRs!

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.

1 participant