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

generated model phpdocs #780

Merged
merged 4 commits into from
May 22, 2019
Merged

generated model phpdocs #780

merged 4 commits into from
May 22, 2019

Conversation

mx2s
Copy link
Contributor

@mx2s mx2s commented May 22, 2019

I've generated model phpdocs with https://github.com/barryvdh/laravel-ide-helper, this should help with IntelliSense in IDE's

@HDVinnie
Copy link
Collaborator

@MXSS Thanks. I also use the IDE helper for phpstorm however never really liked the docs in the models or controllers for the class. Just have been doing the functions themselves. Just seems like alot of bloat. You think its worth it?

@mx2s
Copy link
Contributor Author

mx2s commented May 22, 2019

@HDVinnie Yes, I think it's worth it, if there's too much docs I can remove @method info because @property docs are most helpful. That should reduce docs size in ~half

@HDVinnie HDVinnie merged commit 7b60d56 into HDInnovations:master May 22, 2019
@mx2s mx2s deleted the model-docs branch May 23, 2019 05:49
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