rawilk / laravel-settings

Store Laravel application settings in the database.
https://randallwilk.dev/docs/laravel-settings
MIT License
200 stars 17 forks source link

Lower PHP version support #1

Closed ZBrettonYe closed 4 years ago

ZBrettonYe commented 4 years ago

Love the idea of this package.

Cloud this package require lower PHP version? since the Laravel official framework 7.x PHP version minimum requirement is 7.2.5

rawilk commented 4 years ago

Thanks for your interest in this package. Since I use php 7.4 syntax in the package, such as specifying types for class properties, and using arrow functions (fn () =>), it is not possible to lower the minimum php version.

I personally only use php 7.4, which is why I write my packages using the available language syntax. I see no reason why you wouldn't upgrade your php versions, but I do understand there may be cases where you are limited by your hosting provider or something like that.

rawilk commented 4 years ago

Btw, you are more than welcome to fork and maintain your own version that is compatible with php 7.2 or whatever version you need. Just be sure to include the original license in the repo.

ZBrettonYe commented 4 years ago

Thanks for your speedy respond 👍

I will upgrade my PHP environment to 7.4 for this package, since I will upgrade it one day anyway