Skip to content
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

Update the Yaml driver #39

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

ludofleury
Copy link

As mentioned in the symfony\yaml since 2.2:

"Passing a file as an input is a deprecated feature and will be removed in 3.0."

As mentioned in the [symfony\yaml](symfony/yaml@a436568)
"Passing a file as an input is a deprecated feature and will be removed in 3.0."
@keyboardSmasher
Copy link

GG I was just getting ready to submit a pull request. The error message about the foreach loop didn't help much.

Anyway, it seems Igor has abandoned the repository. :(

@dominikzogg
Copy link

@igorw please merge and tag this, without this fix, the provider does not run on symfony 2.7 components without deprecation messages

@Addvilz
Copy link

Addvilz commented Jun 5, 2015

+1

@keyboardSmasher
Copy link

Igor, in his own words, is now a software philosopher. I suggest forking, patching and using your fork in your project. Hakuna Matata.

@dominikzogg
Copy link

i use my fork, i hope you will join, your PR is allready in an taged as v1.2.3 https://github.com/saxulum/ConfigServiceProvider

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.

4 participants