Update package to support Laravel 6, phpunit 8 and PHP 7.2

vicgonvt commented 5 years ago

If this pull request gets merged it applies the following changes:

bumps the illuminate/support package to 6.0
bumps the phpunit/phpunit to version ~8.0
Laravel 6 requires php to be in version 7.2, so that change also applies
Fixes tests to stop using the @expectedException notation as it will be deprecated in Phpunit 9 and refactors to use the $this->expectException(\InvalidArgumentException::class); instead
updates the testcase to now include the void return type required by phpunit 8

daijongkim commented 4 years ago

@Jaspaul please review & merge this PR

crhayes commented 4 years ago

@Jaspaul may no longer have the ability to merge this. Pinging @JaskirtPooni @obiwongee

vicgonvt commented 4 years ago

Is this repo dead?

Soapbox / laravel-formatter