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

Slug command #208

Closed
wants to merge 2 commits into from
Closed

Conversation

dan-klasson
Copy link

README says to create a pull request against the development branch, but no such branch exist. So I picked master.

@cviebrock
Copy link
Owner

I'll update the README after this to point merge requests against master ... thanks!

I see that the Laravel 5.1 way of declaring command arguments and options is different, but I assume the "old" way works still. I may eventually need to branch out a 5.x and 5.1 version, I guess ... but don't worry about that for now.

However, can you please write a test for this as well? Happy to merge after that!

@cviebrock
Copy link
Owner

Sorry I never merged this earlier. It's a good suggestion!

Any chance I can get you to resubmit the merge against the new 4.0 branch? And add a test, please!

@dan-klasson
Copy link
Author

I tried to. But I had some problems installing the framework. I don't do PHP anymore. I could write something up without running it, but obviously I cannot guarantee it will work.

@cviebrock cviebrock closed this Jun 6, 2016
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.

None yet

2 participants