liip / LiipMonitorBundle

Integrates the LiipMonitor library into Symfony
http://liip.ch
467 stars 103 forks source link

feat: allow symfony 7 #286

Closed Chris53897 closed 9 months ago

Chris53897 commented 9 months ago

CI is blocked by:

Chris53897 commented 9 months ago

Last Update from enlightn/security-checker is from February 2022. Maybe we can remove that in the ci for symfony 7 run, until that is merged.

kbond commented 9 months ago

Maybe we can remove that in the ci for symfony 7 run, until that is merged.

Ak, we can't as it's a hard dependency of laminas/laminas-diagnostics

Chris53897 commented 9 months ago

Ok. I did ping the maintainer now.

kbond commented 9 months ago

See https://github.com/laminas/laminas-diagnostics/pull/73

Chris53897 commented 9 months ago

Looks good now (after PHP-CS-Fixer added support for Symfony 7). Stability should be changed after the first stable release of symfony 7.

And i think this PR https://github.com/liip/LiipMonitorBundle/pull/283 should be merged first. to ensure PHP 8.3 test.

kbond commented 9 months ago

to ensure PHP 8.3 test.

Unfortunately, laminas-diagnostics doesn't support 8.3 yet.

Chris53897 commented 9 months ago

Ok i see. Lets wait for this PR to be merged. https://github.com/laminas/laminas-diagnostics/pull/74

Update: merged, but not yet released.

kbond commented 9 months ago

Is this one ready to merge or do you still want to wait for #283?

Chris53897 commented 9 months ago

Is it ready to merge

kbond commented 9 months ago

Thanks @Chris53897!

alessandro-podo commented 8 months ago

Is there any planning for when a new release will be released with this feature?

kbond commented 7 months ago

Thanks for the ping @alessandro-podo, I just released 2.22.0 which includes Symfony 7 support.