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

Minor Docblock fixes for @param and @return tags #12

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Minor Docblock fixes for @param and @return tags #12

wants to merge 3 commits into from

Conversation

czim
Copy link

@czim czim commented Jun 9, 2016

PHPStorm (and other IDEs) tend to give incorrect warnings due to the incorrect doc blocks. This PR fixes the following:

  • IDEs not understanding Restable fluent syntax (such as for Restable::listing([])->render(), fixed by updating the listing() @return tag).
  • IDEs incorrectly warning that non-array parameters for many methods are not accepted (whereas objects with toArray() methods are supported).

These fixes only affect doc blocks, no functionality or other comments are affected.

@czim
Copy link
Author

czim commented Mar 8, 2017

Not to be rude, but is this project discontinued? This PR literally changes nothing but a few docblocks and has been sitting here for the better part of a year...

If this is an unacceptable update, please let me know why and what needs to be done to fix it.

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.

1 participant