search

Vinelab / cdn

CDN Assets Manager Package for Laravel.
MIT License
216 stars 71 forks source link

Laravel 5.2 and PHP 7 Update #91

Closed publiux closed 8 years ago

publiux commented 8 years ago

This pull request updates the composer dependencies to sync with Laravel 5.2's dependencies. It raises the required PHP version to 5.5.9.

Also, for Travis CI, PHP 7 is no longer an allowed failure.

Suggest creating a new Laravel 5.2 branch for compatibility reasons.

vovanmix commented 8 years ago

Please merge it!

Mulkave commented 8 years ago

@publiux thank you for this PR! Sorry missed this one. Does that grants backward-compatibility with 5.1? In that case we won't have to create a new branch for this one.

Mulkave commented 8 years ago

@publiux any idea if this will remain working for 5.1? @vovanmix did you try it out ?

simplenotezy commented 8 years ago

+1 merge please <3

Mulkave commented 8 years ago

@canfiax would love to, just wondering if it breaks backward compatibility with 5.1. Sorry for taking long on this but I won't have the time to to try things out so if someone can please let me know so that I merge.

Mulkave commented 8 years ago

After taking a keen look at the code, it only breaks compatibility with PHP -5.5.9 which is acceptable. Merging.