Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[FrameworkBundle] Removed deprecated code paths #15695

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
Sep 6, 2015

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