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

Handle PRs with over 100 comments #17

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Gudahtt
Copy link

@Gudahtt Gudahtt commented Sep 10, 2020

When trying to find an existing comment or look for a signature, the bot only looks at the first 100 comments of the PR. This is because 100 is the default page size for PR comments returned by the GitHub API. As a result, if many comments are made before the bot comment, it will keep re-posting it. Additionally, if many comments are made before the signature itself, the bot will never find it.

Both attempts to find comments have been updated to continue asking for pages until all comments have been reviewed.

@Gudahtt Gudahtt force-pushed the support-PRs-with-over-100-comments branch from d1040d4 to fbb317a Compare September 10, 2020 15:39
When trying to find an existing comment or look for a signature, the
bot only looks at the first 100 comments of the PR. This is because 100
is the default page size for PR comments returned by the GitHub API.
As a result, if many comments are made before the bot comment, it will
keep re-posting it. Additionally, if many comments are made before the
signature itself, the bot will never find it.

Both attempts to find comments have been updated to continue asking for
pages until all comments have been reviewed.
@Gudahtt Gudahtt force-pushed the support-PRs-with-over-100-comments branch from fbb317a to 9898a54 Compare September 10, 2020 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant