atoum / reports-extension

atoum reports extension
http://atoum.github.io/reports-extension/
Other
6 stars 6 forks source link

PHP 8 support #51

Closed franck-paul closed 3 years ago

franck-paul commented 3 years ago

It should be great to have PHP 8 for this extension.

Grummfy commented 3 years ago

indeed, sorry but I have no time to work on it now. But we accept PR :D

franck-paul commented 3 years ago

I would try but I don't know how to install the dev environment, composer install --prefer-source seems not working. I have error thrown on unit testing with vendor/bin/atoum --test-ext command:

PHP Fatal error:  Uncaught Error: Class "mageekguy\atoum\reports\asynchronous" not found in /Users/franck/Documents/EGO/Geekeries/Projets/reports-extension/classes/coverage.php:7
Stack trace:

Sorry but I'm a rookie with composer :-)

franck-paul commented 3 years ago

PR submitted.

Grummfy commented 3 years ago

yes, I look at it right now https://github.com/atoum/reports-extension/pull/53

Grummfy commented 3 years ago

atoum/reports-extension version 4.0.0 is ready, compatible with 8.0.0