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

Fix @property name typos in RestClient PHPDoc #229

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

Conversation

ZebulanStanphill
Copy link

@ZebulanStanphill ZebulanStanphill commented Oct 11, 2021

The PHPDoc for the RestClient class had some typos in its @property declarations, with several missing an s at the end and one using the wrong capitalization. For example, messages was incorrectly listed as message, which was quite confusing for me when using my IDE's auto-completion and comparing against this project's official documentation.

This PR fixes those issues.

Copy link
Contributor

@narayana-plivo narayana-plivo left a comment

Choose a reason for hiding this comment

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

LGTM

@narayana-plivo
Copy link
Contributor

LGTM

@huzaif-plivo

@mkhrystunov
Copy link

Hello! can this be pushed and released any time soon?

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.

3 participants