search

ejoffe / spr

Stacked Pull Requests on GitHub
MIT License
796 stars 68 forks source link

fix: support large repos by fetching only user's pull requests #342

Closed Delerme closed 1 year ago

Delerme commented 1 year ago

Previously, all pull requests for the current repo were queried, with a limit of 100. This means spr would not function for any repos with over 100 open pull requests, which is fairly common for larger repos. Instead, query for only the current user's pull requests.

Fixes #319

ejoffe commented 1 year ago

Thank you for this fix! I actually looked into this, and didn't realize that pullRequest can be asked inside the viewer to only get these PRs. I was banging my head trying to find a filter for the user in the github api.

ejoffe commented 1 year ago

Looks like some of the unit tests need to be fixed up for the new api change.

Delerme commented 1 year ago

Looks like some of the unit tests need to be fixed up for the new api change.

Whoops, fixed