Skip to content

Update readme - #35

Merged
marcelgwerder merged 5 commits into
marcelgwerder:masterfrom
jeremykenedy:master
Apr 3, 2017
Merged

marcelgwerder merged 5 commits into
marcelgwerder:masterfrom
jeremykenedy:master

Conversation

@jeremykenedy

Copy link
Copy Markdown
Contributor

Update readme to have direct pull of package instruction from CLI and update call examples for app.php to reflect current Laravel name conventions.

Update readme to have direct pull of package instruction from CLI and update call examples for `app.php`  to reflect current Laravel name conventions.
@marcelgwerder

Copy link
Copy Markdown
Owner

Thanks for your PR, I should have done some changes to the readme regarding installation for a long time.
I think composer require marcelgwerder/laravel-api-handler is sufficient and we don't need anything else especially not the dangerous composer update. Can you remove the rest as part of this PR?

Removed direct composer.json editing instruction per authors suggestion :)
@jeremykenedy

Copy link
Copy Markdown
Contributor Author

Cool, I have updated that. Thank you for the package!

@marcelgwerder
marcelgwerder merged commit c1d5a9b into marcelgwerder:master Apr 3, 2017
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.

2 participants