-
Notifications
You must be signed in to change notification settings - Fork 14
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
Allow Symfony 2.8 #1
Conversation
1485439
to
0dddc14
Compare
https://travis-ci.org/Algatux/influxdb-bundle/jobs/134418403#L171 Seems some of your dependencies are not compatible with Symfony 3.1. Shall I remove this part? Related issue: phpspec/prophecy#272 |
Hi thanks a lot for your pr. I will take a look as soon as possible. Yes please remove symfony 3.1 for now. |
As a library, dependencies tests should not be locked at all.
Symfony 2.8 IS the same as 3.0, but with the deprecated old feature. This was done to provide an easy upgrading way between 2.x and 3.x. Lot of people are still on 2.x version of Symfony. Nothing has to be changed on the project codebase. Plus, this is a LTS version. It's important to keep support on it. Ref: http://symfony.com/blog/transition-from-symfony-2-7-to-3-0-symfony-2-8-on-its-way
The goal is to test the library against multiple versions of Symfony and with lowest dependencies.
@Algatux done! 👍 |
Thanks you for the merge! 👍 Could you please tag a new release for that? |
@soullivaneuh @greg0ire sorry for the squash guys! I was verry excited for the PR 😇 |
Symfony 2.8 IS the same as 3.0, but with the deprecated old feature.
This was done to provide an easy upgrading way between 2.x and 3.x.
Lot of people are still on 2.x version of Symfony.
Nothing has to be changed on the project codebase.
Plus, this is a LTS version. It's important to keep support on it.
Ref: http://symfony.com/blog/transition-from-symfony-2-7-to-3-0-symfony-2-8-on-its-way
Note I also removed the
composer.lock
file and replacedcomposer install
bycomposer update
.As a library, dependencies tests should not be locked at all.
Travis was update to test multiple Symfony versions so you will have the proof that your bundle is compatible. 👍