New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add missing fields to PR stream #49

Open

Ry-DS opened this issue Nov 18, 2021 · 1 comment

Contributor

Ry-DS commented Nov 18, 2021 •

edited

Loading

Currently the PR stream is missing a bunch of key fields

additions
deletions
review_comments (int)
changed_files (int)
..etc

The current api route we use https://docs.github.com/en/rest/reference/pulls#list-pull-requests isn't sufficient to get all these missing fields. We would need to use the singular pr route https://docs.github.com/en/rest/reference/pulls#get-a-pull-request to fetch everything. This may be difficult since this would greatly increase the amount of requests made to the API. We could maybe cut this down by making a custom query with absolutely everything with #48.

The text was updated successfully, but these errors were encountered:

Ry-DS mentioned this issue

Implement a base GraphQL stream to reduce unnecessary API calls #48

Closed

Contributor Author

Ry-DS commented Mar 23, 2022

Won't be possible without pagination #89

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment