merged-pr-stat icon indicating copy to clipboard operation
merged-pr-stat copied to clipboard

PR count hard limit of 1000

Open nickrobillard opened this issue 1 year ago • 1 comments

In high PR volume repos, I am seeing a hard limit of 1000 count. After some searching, I do see references to a Github GraphQL API limit of 1000. I guess paging is the answer here? I can work around it by shortening the time period to get the count below 1000 and then combine them, but this is quite painful.

This example is from a Github Enterprise repo, in case that makes a difference.

Example of 1000 limit response:

{
  "count": 1000, <- there are more than 1000
  "authorCount": 31,
  "additions": 509202,
  "additionsAverage": 509.202,
  "additionsMedian": 21,
  "deletions": 969856,
  "deletionsAverage": 969.856,
  "deletionsMedian": 11,
  "leadTimeSecondsAverage": 178136,
  "leadTimeSecondsMedian": 11832,
  "timeToMergeSecondsAverage": 59420,
  "timeToMergeSecondsMedian": 4366,
  "timeToMergeFromFirstReviewSecondsAverage": 59379,
  "timeToMergeFromFirstReviewSecondsMedian": 4317
}

nickrobillard avatar Oct 26 '23 22:10 nickrobillard

If you have more than 1,000 entries, it would be better to have automatic paging to retrieve all the entries

shibayu36 avatar Oct 27 '23 01:10 shibayu36