influxdb-bundle

influxdb-bundle copied to clipboard

Support PHP 8

2

Currently the bundle can't be used on PHP 8 because of the version constraint. Please consider supporting PHP 8 as well. From a quick check your dependencies all support PHP...

TPete

chore(ci): Make the CI happy

6

Using `SYMFONY_REQUIRE` and `symfony/flex` for specifying the Symfony version to use, see https://github.com/symfony/flex/pull/409. We gain ~10 secs during the `composer update` (https://travis-ci.org/github/Algatux/influxdb-bundle/builds/671260308 vs https://travis-ci.org/github/Algatux/influxdb-bundle/builds/740050252)

Kocal

Including phpunit ad dev dependency causes CI to broke

6

Including phpunit ad dev dependency causes CI to broke when the --prefer-lowest composer flag is applied see dependecies_update branch

Algatux

Queries profiler

9

It would be a very great addition to have Influx-db queries listing on the profiler. Like Doctrine does:   What do you think?

soullivaneuh

Bundle class should be vendor prefixed

1

It should be `AlgatuxInfluxDbBundle` and not just `InfluxDbBundle` to reduce possible bundle conflicts. This is the adopted way for 99% of external bundles as you can see on this `AppKernel`...

soullivaneuh

Update docker dev environment

3

We need to update the local dev environment

Algatux

# Summary Ideally, with a transitional versions supporting both v1 and v2. Doc: https://docs.influxdata.com/influxdb/v2/

soullivaneuh

Symfony 7 support will be more tricky because of the removal of the container aware trait. It make the needed refactor a bit more complex because of the called of...

soullivaneuh