Drop Symfony 2.8 support, add Symfony 5 support, set PHP requirements to ^7.2

[Kocal]

This PR is a proposal for Symfony 5.0 support.

Also, I've seen issue #89 and #88 while working on this PR. I think it's ok to drop support of Symfony 2.8 and up PHP requirements to PHP 7.1 here, but I can remove those commits if needed. :+1:

EDIT: we have some issues on Travis:

• matthiasnoback/symfony-dependency-injection-test 4.1.0 is required for Symfony 5 support, but it requires PHP 7.2 (https://travis-ci.org/Algatux/influxdb-bundle/jobs/648326614, https://travis-ci.org/Algatux/influxdb-bundle/jobs/648326615, https://travis-ci.org/Algatux/influxdb-bundle/jobs/648326616, https://travis-ci.org/Algatux/influxdb-bundle/jobs/648326617)
• phpunit/phpunit is not installable due to conflicts with matthiasnoback/symfony-dependency-injection-test and matthiasnoback/symfony-config-test (https://travis-ci.org/Algatux/influxdb-bundle/jobs/648326618, https://travis-ci.org/Algatux/influxdb-bundle/jobs/648326619, https://travis-ci.org/Algatux/influxdb-bundle/jobs/648326620)

What do you think of:

• remove matthiasnoback/symfony-dependency-injection-test
• remove the composer require "phpunit/phpunit:^5.6" --no-update line, which I think is useless since symfony/phpunit-bridge alreayd install PHPUnit
• use nyholm/symfony-bundle-test which:
  • requires php ^5.5 or ^7.0
  • does not require or conflict with phpunit/phpunit
  • have support for Symfony ^3.4, ^4.3 and ^5.0
  • but will require a lot of work

Another solution is to set PHP requirement to ^7.2 which is still maintained unlike 7.1: https://www.php.net/supported-versions.php

[Algatux]

@Kocal thanks for your contribution! I will look at it asap

[Algatux]

@Kocal about the php issue I was thinking about bumping the requirement up to 7.2 because 7.1 reached its end of life. What do you think @soullivaneuh ?

Still looking at other issues

[Kocal]

Hi, is there any updates on this? I think I will go for PHP 7.2 as minimum requirement

[Kocal]

CI is nearly green everywhere except for Symfony 4.0 :/

I don't really have time to investigate the issue, I will now use my fork for SF5 support.

Thanks

[tristanbes]

thank you @Kocal

[Algatux]

@Kocal because of the bc introduced by bumping up the requirements I have to release a new major version. I think I will create a new 3.0 beta to test it and fix the test suite issues

[Kocal]

Great, thanks you!

[tristanbes]

@Algatux did you had the chance to address: https://github.com/Algatux/influxdb-bundle/pull/92#issuecomment-609420066 ?

[Kocal]

Just for information, we are using the bundle in production since ~5 months and everyhing works fine.

Is there something missing for a new major release?

[Algatux]

Hi @Kocal. Before the release, we need to fix the CI to be sure that everything works in every expected environment. Actually, I do not have much time to do so.

If you have some time to investigate, please feel free to open an MR or a discussion on how to proceed to fix these issues.

Thanks a lot

[Kocal]

Alright, I will take a look ASAP! :)