New issue
Advanced search Search tips

Issue 740175 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Aug 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Feature

Blocking:
issue 707006



Sign in to add a comment

[WPT Export] Implement pagination in WPTGitHub.all_pull_requests

Project Member Reported by jeffcarp@google.com, Jul 7 2017

Issue description

Currently we only fetch the 100 most recent pull requests. We should fetch all of them.
 
Labels: -Type-Bug Type-Feature
Project Member

Comment 2 by sheriffbot@chromium.org, Jul 11 2017

Labels: Hotlist-Google
Cc: jeffcarp@chromium.org
Owner: robertma@chromium.org
Status: Started (was: Assigned)
By the way, do we also need pagination for WPTGitHub.in_flight_pull_requests? As a side note, there are currently 29 open PRs with the chromium-export label, and the default item number per page is 100 for search results (30 for other GitHub endpoints).
It looks like there actually is no longer anything using in_flight_pull_requests, so feel free to delete that method in your CL, or I can create a separate CL for that.
Project Member

Comment 6 by bugdroid1@chromium.org, Aug 2 2017

Status: Fixed (was: Started)

Sign in to add a comment