Skip to content

Conversation

fabpot
Copy link
Member

@fabpot fabpot commented Oct 8, 2016

Q A
Branch? master
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets n/a
License MIT
Doc PR n/a

HttpKernel does not have console as a dependency, so we must ensure that this is not implicitly required (DebugHandlersListener already has this check).

@nicolas-grekas
Copy link
Member

👍

@fabpot fabpot merged commit 414206d into symfony:master Oct 8, 2016
fabpot added a commit that referenced this pull request Oct 8, 2016
This PR was merged into the 3.2-dev branch.

Discussion
----------

[HttpKernel] removed implicit dep on Console

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | n/a
| License       | MIT
| Doc PR        | n/a

HttpKernel does not have console as a dependency, so we must ensure that this is not implicitly required (`DebugHandlersListener` already has this check).

Commits
-------

414206d [HttpKernel] removed implicit dep on Console
@fabpot fabpot deleted the console-dep-removal branch October 24, 2016 21:39
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