search

Alymosul / laravel-exponent-push-notifications

Exponent push notifications driver for laravel 5
MIT License
144 stars 77 forks source link

Added support for Laravel 6.0 #35

Closed pelmered closed 5 years ago

Alymosul commented 5 years ago

I think the travis script needs some changes..

pelmered commented 5 years ago

@Alymosul Yes. The version of PHP is incompatible with the version of PHPUnit included.

I would suggest that we only use PHP 7.2 and 7.3 in the Travis test matrix. I can fix that if you want.

pelmered commented 5 years ago

Fixed both Travis and Scrutinizer scripts.

Required PHP-version is now 7.1.3. That is the minimum requirement for Laravel 5.6+. Laravel 6.0 has a minimum requirement of 7.2 and I think the requirement for this package should probably be bumped up to 7.2 as well.

Alymosul commented 5 years ago

Thanks!

pelmered commented 5 years ago

@Alymosul Could you release a new stable version with this fix?