Skip to content
This repository has been archived by the owner on Nov 24, 2023. It is now read-only.

Laravel 5 support #27

Merged
merged 2 commits into from
Mar 24, 2015
Merged

Laravel 5 support #27

merged 2 commits into from
Mar 24, 2015

Conversation

stayallive
Copy link
Contributor

No description provided.

@machuga
Copy link
Owner

machuga commented Feb 19, 2015

Failing due to PHP 5.3 Incompatibility

@stayallive
Copy link
Contributor Author

Oh right, we would have to bump this to 5.4 minimum since L5 does this... any problems with that?

@ubald
Copy link

ubald commented Mar 20, 2015

Yes please update and merge, I also had to fork in order to install.

@machuga
Copy link
Owner

machuga commented Mar 23, 2015

As-is I cannot merge it since it's breaking the build. 2.x supports 5.3, but 3.0 won't have to.

@machuga
Copy link
Owner

machuga commented Mar 23, 2015

@stayallive if you bump the version number to 5.4.0, update the travis.yml file to remove 5.3 and add whatever new php versions there are, I'll merge this in and mark it as a minor version release. Since you're the one motivating me to make this happen I want to make sure you get the credit in the commit history. :)

@stayallive
Copy link
Contributor Author

I have updated the travis file to use the one Laravel uses, hope that's cool with you :)

machuga added a commit that referenced this pull request Mar 24, 2015
@machuga machuga merged commit 1c927c7 into machuga:master Mar 24, 2015
@machuga
Copy link
Owner

machuga commented Mar 24, 2015

Thanks!

@fer-ri
Copy link

fer-ri commented Apr 18, 2015

Cooll.. L5 welcome 🍡

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

Successfully merging this pull request may close these issues.

4 participants