bugsnag / bugsnag-symfony

BugSnag notifier for the Symfony PHP framework. Monitor and report errors in your Symfony apps.
https://docs.bugsnag.com/platforms/php/symfony
MIT License
43 stars 21 forks source link

Add PHP 8 to CI #116

Closed imjoehaines closed 3 years ago

imjoehaines commented 3 years ago

Goal

Add PHP 8 to CI for Symfony 3.4, 4.4, 5 and pre-release versions

This required a small change to tests as PHP 8 introduces named arguments, which we are inadvertently using by calling ReflectionClass::newInstanceArgs. This calls the constructor by unpacking the given argument array and if that array has string keys, it now uses them as named arguments. Our array keys didn't quite match the actual argument names, so this errored without the change

No actual code changes were required