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

Phpdocs cleanup #2

Merged
merged 2 commits into from
Mar 18, 2013
Merged

Phpdocs cleanup #2

merged 2 commits into from
Mar 18, 2013

Conversation

komapa
Copy link
Contributor

@komapa komapa commented Mar 18, 2013

Fixed some of the methods @param PHPDocs (did not match the method signature). Also added some new lines for readability. @seejohnrun, feel free to only incorporate the actual fixes if you think the extra new lines are not helping ;)

It is related to pull request #1 from Igor but you should not be requiring (especially without require_once) vendor/autoload.php as this is supposed to be handled outside of the Tumblr API classes.

* @param string $reblogKey the reblog_key of the post
*
* @return array the response array
*/
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why de-dent this?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It was a mistake, I noticed it last night but forgot to fix it. Not intentional.

seejohnrun added a commit that referenced this pull request Mar 18, 2013
@seejohnrun seejohnrun merged commit 68ed82d into master Mar 18, 2013
@seejohnrun seejohnrun deleted the phpdocs_cleanup branch March 18, 2013 14:38
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