Skip to content
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.

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

Make changelog tool slower to work around inconsistencies in API results #876

Merged
merged 2 commits into from Oct 12, 2022

Conversation

ludoo
Copy link
Collaborator

@ludoo ludoo commented Oct 12, 2022

The GH API sometimes don't return all pulls when using filters, and/or don't sort them properly. This makes it impossible to process pulls in streaming mode, and forces us to fetch them all and sort later. The tool becomes slower but we avoid updating skipping requests when updating CHANGELOG.

@ludoo ludoo added the on:tools New or changed tool label Oct 12, 2022
@ludoo ludoo requested a review from juliocc October 12, 2022 12:25
@ludoo ludoo changed the title Make changelog tool slower to work around inconcistencies in API results Make changelog tool slower to work around inconsistencies in API results Oct 12, 2022
@ludoo ludoo merged commit 162268a into master Oct 12, 2022
@ludoo ludoo deleted the ludo/changelog-api-fix branch October 12, 2022 12:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
on:tools New or changed tool
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant