You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
contentThis issue or pull request belongs to the Docs Content teamecosystemThis issue or pull request should be reviewed by the Docs Ecosystem teamrestContent related to rest - overview.
What part(s) of the article would you like to see updated?
Remove per_page and page parameters
Additional information
I didn't verify that the parameters do not have any effect, because it's rather hard to setup a pull request with multiple review requests without bothering anyone else. If you could check in with the API team I'd appreciate it
The text was updated successfully, but these errors were encountered:
janiceilene
added
content
This issue or pull request belongs to the Docs Content team
ecosystem
This issue or pull request should be reviewed by the Docs Ecosystem team
and removed
triage
Do not begin working on this issue until triaged by the team
labels
Jan 4, 2021
Thanks for flagging this issue and for including the link to the detailed information in the Octokit repository. 🙇🏻♀️
I checked with the ecosystem API team and they agree with you. The source files for the API docs are maintained in a different repository, so I'm going to open an internal issue for this change and close this.
@felicitymay the docs still include the per_page and page parameters. Will these be removed eventually? I still have an open issue at octokit/rest.js#33 and I'd like to give the Octokit user an update on their question
contentThis issue or pull request belongs to the Docs Content teamecosystemThis issue or pull request should be reviewed by the Docs Ecosystem teamrestContent related to rest - overview.
Follow up to https://github.com/octokit/rest.js/issues/1978 /cc @nikclayton-dfinity
What article on docs.github.com is affected?
https://docs.github.com/en/free-pro-team@latest/rest/reference/pulls#list-requested-reviewers-for-a-pull-request
What part(s) of the article would you like to see updated?
Remove
per_page
andpage
parametersAdditional information
I didn't verify that the parameters do not have any effect, because it's rather hard to setup a pull request with multiple review requests without bothering anyone else. If you could check in with the API team I'd appreciate it
The text was updated successfully, but these errors were encountered: