Skip to content

Conversation

@ghost
Copy link

@ghost ghost commented Sep 26, 2016

I've been using this fork for 3 weeks now and it works just fine.

Please merge this so that Symfony 3 support is added.

@ghost
Copy link
Author

ghost commented Sep 28, 2016

@bezhermoso Please merge this so that we can use this on Symfony 3 without having to mess around with composer.

@vsmoraes
Copy link

👍 we need this

@IlyaPokamestov
Copy link

@bezhermoso @jwestman-activelamp @tomfriedhof guys what should be done to merge this one? I can continue this PR if that needed.
Thanks!

@bezhermoso
Copy link
Contributor

Hi all,

I haven't checked up on this repo for quite a while now. I will take a closer look at issues/PRs early December; I'd like to get this upgraded for Symfony 3 as well. This might have to wait until after we get the Travis builds in place and have the test pass again. What do you guys think?

@vsmoraes
Copy link

Awesome! I'm working on a brand new project and soon this feature will be a necessary. You can count on me if any kind of help is needed o/

@ghost
Copy link
Author

ghost commented Dec 5, 2016

@bezhermoso It would be good to have Travis in place, but this merge is pretty important in my opinion. I've been using this fork for a while now, but I prefer to use the original repo (which I can only do once this PR has been merged / this project has been upgraded to Symfony3).

@bezhermoso bezhermoso merged commit 6f3af3a into activelamp:master Dec 16, 2016
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