8000 [TwigBundle] do not remove the Twig ExceptionController service by xabbuh · Pull Request #20974 · symfony/symfony · GitHub
[go: up one dir, main page]
Skip to content

[TwigBundle] do not remove the Twig ExceptionController service #20974

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Dec 17, 2016
Merged

[TwigBundle] do not remove the Twig ExceptionController service #20974

merged 1 commit into from
Dec 17, 2016

Conversation

xabbuh
Copy link
Member
@xabbuh xabbuh commented Dec 17, 2016
Q A
Branch? 2.8
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets
License MIT
Doc PR follow up of #20799

@fabpot
Copy link
Member
fabpot commented Dec 17, 2016

Thank you @xabbuh.

@fabpot fabpot merged commit c9e560f into symfony:2.8 Dec 17, 2016
fabpot added a commit that referenced this pull request Dec 17, 2016
@fabpot
minor #20974 [TwigBundle] do not remove the Twig ExceptionController …
2ef2264 
…service (xabbuh)

This PR was merged into the 2.8 branch.

Discussion
----------

[TwigBundle] do not remove the Twig ExceptionController service

| Q             | A
| ------------- | ---
| Branch?       | 2.8
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets |
| License       | MIT
| Doc PR        | follow up of #20799

Commits
-------

c9e560f do not remove the Twig ExceptionController service
@xabbuh xabbuh deleted the fix-twig-controller branch December 17, 2016 22:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Bug Status: Needs Review TwigBundle
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@xabbuh @fabpot @carsonbot
0