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

Updated illuminate dependencies for ~5.0 #261

Closed
wants to merge 1 commit into from
Closed

Conversation

simplenotezy
Copy link

Please merge this into main repo. This ensures compatibility for Laravel 5.

Currently L5 composer update fails.

@simplenotezy
Copy link
Author

Status on this?

@igorsantos07
Copy link
Member

As this would make new releases incompatible with Laravel 4, there's a question that needs to be answered: is the current codebase REALLY compatible with Laravel 5? This needs to be tested before releasing it - or, maybe, we can release as a new major version.

@@ -25,9 +25,9 @@
},
"require": {
"php": ">=5.3.0",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Include here PHP 5.4 as well. That's a new requirement for Laravel 5.

@davincho
Copy link

@igorsantos07 Any plans when you are going to support Laravel 5? Happy to help ....

@igorsantos07
Copy link
Member

@davincho I was going to accept this PR into another branch, for testing, but the author didn't replied back my comments. As I'm probably going to need Ardent soon, I might as well tackle this myself in the following weeks, tho.

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

Successfully merging this pull request may close these issues.

4 participants