Skip to content

Conversation

@saro0h
Copy link
Contributor

@saro0h saro0h commented Dec 26, 2014

Q A
Bug fix? no
New feature? no
BC breaks? yes
Deprecations? no
Tests pass? yes
Fixed tickets ~
License MIT
Doc PR ~

Choose a reason for hiding this comment

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

we should remove ApacheUrlMatcher also?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes sure.

@timglabisch
Copy link

👍 but this PR could also remove ApacheMatcherDumper + ApacheUrlMatcher

@saro0h saro0h force-pushed the remove-router-apache-dumper branch from 3837e9b to 2a9bce5 Compare December 28, 2014 17:18
@saro0h saro0h force-pushed the remove-router-apache-dumper branch from 2a9bce5 to b552b1b Compare December 28, 2014 17:20
@fabpot
Copy link
Member

fabpot commented Dec 30, 2014

Thank you @saro0h.

@fabpot fabpot merged commit b552b1b into symfony:master Dec 30, 2014
fabpot added a commit that referenced this pull request Dec 30, 2014
…umperCommand (saro0h)

This PR was merged into the 3.0-dev branch.

Discussion
----------

[FrameworkBundle] Removed the deprecated RouterApacheDumperCommand

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | yes
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | ~
| License       | MIT
| Doc PR        | ~

Commits
-------

b552b1b [FrameworkBundle] Removed the deprecated RouterApacheDumperCommand
@fabpot fabpot mentioned this pull request Nov 16, 2015
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.

3 participants