-
Notifications
You must be signed in to change notification settings - Fork 217
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Why php > 7.1 requirement and not > 7.0? #71
Comments
Hi @blomdahldaniel! It's always tough to decide how to determine dependency restrictions. In this case I always determine required PHP version by the highest PHP version required by compatible Laravel versions or dependencies. In this case it is Laravel 5.6 that requires PHP 7.1.3. :) Also, I try to only support active PHP versions (7.0.x only receives security updates anymore and is on its way out, see here: https://secure.php.net/supported-versions.php). I'm glad to hear that you like the package though! And thanks for reaching out. :) |
Thanks for a great response and that you clarified this. Thanks again for
sharing your work!
Den ons 14 feb. 2018 00:52Mike Bronner <notifications@github.com> skrev:
… Closed #71 <#71>.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#71 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AJlCva3pmUYaYDXPw5xez63W1miQ5qYDks5tUiBVgaJpZM4SEezO>
.
|
Dude! This is so great!
I have created something similar myself in two different projects but this solves it in a much cleaner way. Glad to see your solutions! 😃
I wonder though, why do you force php 7.1? Why not go for the broader audience and allow also php 7.0?
Thanks again for sharing your stuff!
The text was updated successfully, but these errors were encountered: