8000 [FrameworkBundle] Removed deprecated code paths by nicolas-grekas · Pull Request #15695 · symfony/symfony · GitHub
[go: up one dir, main page]

Skip to content

Conversation

@nicolas-grekas
Copy link
Member
Q A
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets -
License MIT
Doc PR -

@Tobion
Copy link
Contributor
Tobion commented Sep 4, 2015

validator_api_version still there

@dosten
Copy link
Contributor
dosten commented Sep 4, 2015

You should fix the error in Symfony/Bridge/ProxyManager/LazyProxy/PhpDumper/ProxyDumper

@nicolas-grekas
Copy link
Member Author

@Tobion yes, to be removed in an other PR

@nicolas-grekas
Copy link
Member Author

@dosten which error??

@dosten
Copy link
Contributor
dosten commented Sep 5, 2015

@nicolas-grekas Hmm.. It's weird, a few hours ago the cause of the failing job on Travis was the use of the SCOPE_CONTAINER constant in the Symfony/Bridge/ProxyManager/LazyProxy/PhpDumper/ProxyDumper class, but now seems to be ok.

👍

@nicolas-grekas
Copy link
Member Author

Status: needs work
I need to figure out what's wrong with the test.
It may be related to the per-component testing system.

@nicolas-grekas
Copy link
Member Author

Failure inspected, will be fixed once #15694 is merged.
Status: needs review

@fabpot
Copy link
Member
fabpot commented Sep 6, 2015

Thank you @nicolas-grekas.

@fabpot fabpot merged commit 640b95a into symfony:master Sep 6, 2015
fabpot added a commit that referenced this pull request Sep 6, 2015
…as-grekas)

This PR was merged into the 3.0-dev branch.

Discussion
----------

[FrameworkBundle] Removed deprecated code paths

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

Commits
-------

640b95a [FrameworkBundle] Removed deprecated code paths
@nicolas-grekas nicolas-grekas deleted the fb30 branch September 6, 2015 15:00
@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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

0