Skip to content

Removed illuminate/foundation as a requirement #63

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

Merged
merged 2 commits into from
May 29, 2015

Conversation

frusdelion
Copy link

I have found out today that illuminate/foundation has been abandoned.
I propose a workaround that imports the entire framework, this provides the same foundation code that illuminate/foundation had. This is to ensure that anyone who still uses laravel 4.* will be able to deploy this package to their projects.

illuminate/foundation has been deprecated. Proposed a workaround that imports the entire framework.
"illuminate/foundation": "4.*",
"illuminate/support": "4.*"
"illuminate/support": "4.*",
"laravel/framework": "4.*"
Copy link
Contributor

Choose a reason for hiding this comment

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

We don't need this, just the support dependency will do.

@GrahamCampbell
Copy link
Contributor

👍

@GrahamCampbell
Copy link
Contributor

This is needed for anyone using newer versions of L4.

jeremeamia added a commit that referenced this pull request May 29, 2015
Removed illuminate/foundation as a requirement
@jeremeamia jeremeamia merged commit 9f9633f into aws:1.0 May 29, 2015
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.

3 participants