remove ListOptions from RepositoryListAllOptions (they have no effect) by beyang · Pull Request #751 · google/go-github · GitHub
Skip to content

remove ListOptions from RepositoryListAllOptions (they have no effect)#751

Merged
dmitshur merged 2 commits into
google:masterfrom
beyang:bl/fix-opt
Oct 13, 2017
Merged

remove ListOptions from RepositoryListAllOptions (they have no effect)#751
dmitshur merged 2 commits into
google:masterfrom
beyang:bl/fix-opt

Conversation

@beyang

@beyang beyang commented Oct 13, 2017

Copy link
Copy Markdown
Contributor

The standard pagination options have no effect in the /repositories endpoint, both empirically and according to the documentation: https://developer.github.com/v3/repos/#list-all-public-repositories

Note: Pagination is powered exclusively by the since parameter. Use the Link header to get the URL for the next page of repositories.

@googlebot googlebot added the cla: yes Indication that the PR author has signed a Google Contributor License Agreement. label Oct 13, 2017

@dmitshur dmitshur left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Confirmed. Even the per_page option has no effect.

LGTM. Thanks!

We'll need to bump libraryVersion up by one, since this is a breaking API change:

Feel free to do that, otherwise we can do it before merging ourselves.

@dmitshur dmitshur requested a review from gmlewis October 13, 2017 20:05

@gmlewis gmlewis left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM. Thank you @beyang and @shurcooL!

@dmitshur

Copy link
Copy Markdown
Member

I'll bump libraryVersion and merge this.

@googlebot

Copy link
Copy Markdown

@googlebot googlebot added cla: no and removed cla: yes Indication that the PR author has signed a Google Contributor License Agreement. labels Oct 13, 2017
@dmitshur dmitshur merged commit bf5bc4e into google:master Oct 13, 2017
nbareil pushed a commit to nbareil/go-github that referenced this pull request May 1, 2018
The standard pagination options have no effect in the /repositories endpoint,
both empirically and according to the documentation:

> Note: Pagination is powered exclusively by the since parameter. Use the
> Link header to get the URL for the next page of repositories.

GitHub API docs: https://developer.github.com/v3/repos/#list-all-public-repositories

Bump up libraryVersion since this is a breaking API change.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants