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: minor bugs and doc update #1116

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

Conversation

haridarshan
Copy link
Contributor

Fixes:

  • Add array parameter to repositories method of CurrentUser and User Api to pass additional parameters like per_page and makes the method future proof if GitHub decides to add more options and deprecate existing arguments in support of this new argument. (Ref: Why defaults don't match the github doc? #799 (comment))

Doc:

  • Update repositories method in CurrentUser and User doc
  • Add PR Merge in doc

Closes #1057, Closes #866, Closes #799

…tories, `PSR-18` clients and `HTTPlug` factories

Signed-off-by: Hari Darshan Gorana <er.haridarshan@gmail.com>
Signed-off-by: Hari Darshan Gorana <er.haridarshan@gmail.com>
…tories` method and deprecate other arguments.

Signed-off-by: Hari Darshan Gorana <er.haridarshan@gmail.com>
Signed-off-by: Hari Darshan Gorana <er.haridarshan@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant