Skip to content
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

Update to Laravel 6 & 7, + PHP 7.4 compatible #93

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

lauhakari
Copy link

This might be nice to have.

What I could notice no big changes and tests passed.

@ben182
Copy link

ben182 commented Sep 6, 2019

Addressing #94

@mnvx
Copy link

mnvx commented Sep 19, 2019

Travis fails on PHP 5.6. Maybe will be better to just remove 5.6 from Travis config, because Laravel 6 supports PHP 7.2+

Updated php-versions.
Fixed PHP 7.4  compatibility, for: ErrorException: Unparenthesized `a ? b : c ? d : e` is deprecated
Added Travis CI badge.
Added Travis CI badge.
@lauhakari lauhakari changed the title Update to be Laravel 6 compatibility Update to Laravel 6 & PHP 7.4 compatible Mar 9, 2020
@lauhakari lauhakari mentioned this pull request May 4, 2020
@weidmaster
Copy link

Merging this does fix the issue, as it is only a parenthesis error in the ternary condition. Please accept it so we can update in our composer json. Thank you.

@lauhakari lauhakari changed the title Update to Laravel 6 & PHP 7.4 compatible Update to Laravel 6 & 7, + PHP 7.4 compatible Jun 5, 2020
@vincentkedison
Copy link

Please help to release it to packagist.
I use ben182/laravel-auto-translate, and this package depends on themsaid/laravel-langman.
Causing error on php 7.4

ErrorException  : Unparenthesized `a ? b : c ? d : e` is deprecated. Use either `(a ? b : c) ? d : e` or `a ? b : (c ? d : e)`

Thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants