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 Laravel Recipe #392

Closed
wants to merge 1 commit into from
Closed

Update Laravel Recipe #392

wants to merge 1 commit into from

Conversation

KennedyTedesco
Copy link
Contributor

The current stable version of Laravel (5.1) doesn't use the vendor folder anymore (to write things). Also added the deploy:writable task.

The current stable version of Laravel (5.1) doesn't use the vendor folder anymore (to write things). I also added the `deploy:writable` task.
@antonmedv
Copy link
Member

What about users who using 5.0?

@KennedyTedesco
Copy link
Contributor Author

In that case an option is have more recipes (laravel5, laravel5.1). Or people can change the writable_dirs by themselves.

@oanhnn
Copy link
Contributor

oanhnn commented Jul 13, 2015

👍
But we need a note for support laravel 4.x and 5.0.x. Can you do it?

@KennedyTedesco
Copy link
Contributor Author

@oanhnn Are you in favor to maintain just one recipe plus note for support other versions?

@oanhnn
Copy link
Contributor

oanhnn commented Jul 15, 2015

@KennedyTedesco: Yes. I think this recipe should support laravel 5.1.x (long term support version) and add a note to support other version. Can you create a pull request to deployphp/docs?
@Elfet: What do you think?

@KennedyTedesco
Copy link
Contributor Author

@oanhnn Yes, I can do that.

@oanhnn
Copy link
Contributor

oanhnn commented Jul 16, 2015

@KennedyTedesco Please create a pull request for docs to deployphp/docs

@antonmedv
Copy link
Member

@KennedyTedesco any news? :)

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