search

ArielMejiaDev / larapex-charts

A Laravel wrapper for apex charts
https://larapex-charts.netlify.app/
MIT License
284 stars 84 forks source link

Latest version requires php 8 #80

Closed snapey closed 8 months ago

snapey commented 1 year ago

The package contains code that requires php8 but composer.json still accepts php 7.4

https://github.com/ArielMejiaDev/larapex-charts/blob/21dc8bfd8f9577ac51e5031de643aa7130679771/src/LarapexChart.php#L28

gives ParseError syntax error, unexpected '|', expecting variable

thinkverse commented 1 year ago

Related discussion on Laracasts: https://laracasts.com/discuss/channels/laravel/syntax-error-unexpected-expecting-variable-t-variable-1

ArielMejiaDev commented 8 months ago

Thanks for reporting the issue, I am going to add a patch release soon