Skip to content

[Console] Remove deprecated code #31689

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
May 30, 2019

Conversation

chalasr
Copy link
Member

@chalasr chalasr commented May 29, 2019

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

@chalasr chalasr added this to the 5.0 milestone May 29, 2019
@chalasr chalasr force-pushed the console-drop-deprecated branch from 491520a to 3dfeb69 Compare May 29, 2019 13:04
@chalasr chalasr force-pushed the console-drop-deprecated branch 4 times, most recently from 04c38c0 to d47e984 Compare May 29, 2019 15:56
@chalasr chalasr force-pushed the console-drop-deprecated branch 2 times, most recently from a90e84c to b2e76fa Compare May 29, 2019 23:05
@chalasr
Copy link
Member Author

chalasr commented May 30, 2019

Failing deps=high build should be fixed when merging.
Ready

nicolas-grekas added a commit that referenced this pull request May 30, 2019
…asr)

This PR was merged into the 4.4 branch.

Discussion
----------

[HttpKernel] Make DebugHandlersListener internal

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

Required to remove the legacy `Event` argument type declaration from its `configure()` method in 5.0.
Unlocks #31689

Commits
-------

1a8db29 [HttpKernel] Make DebugHandlersListener internal
@chalasr chalasr force-pushed the console-drop-deprecated branch from b2e76fa to 317b80f Compare May 30, 2019 13:36
@chalasr
Copy link
Member Author

chalasr commented May 30, 2019

Now green thanks to #31734

@nicolas-grekas
Copy link
Member

Thank you @chalasr.

@nicolas-grekas nicolas-grekas merged commit 317b80f into symfony:master May 30, 2019
nicolas-grekas added a commit that referenced this pull request May 30, 2019
This PR was merged into the 5.0-dev branch.

Discussion
----------

[Console] Remove deprecated code

| 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

Commits
-------

317b80f [Console] Remove deprecated features
@chalasr chalasr deleted the console-drop-deprecated branch May 30, 2019 14:00
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