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

fix: Respect all comments in lastComment validator and comment action #755

Merged
merged 1 commit into from
Jun 20, 2024

Conversation

andekande
Copy link
Contributor

The GitHub API only returns 30 issue_comments by default. That can be problematic when the bot runs on PRs or Issues with a long discussion thread, since it won't be able to find the lastComment of a human or himself.

@shine2lay
Copy link
Member

@andekande there are merge conflicts

@andekande andekande force-pushed the fetch-all-comments branch from 95ba53f to 38576d5 Compare June 20, 2024 01:39
@andekande
Copy link
Contributor Author

Conflicts are expected. I'll happily take care of it.
Would be nice if the PRs could be processed soon.

One unrelated remark: The latest changes to the build pipeline seem to have broken the Coverage reporting that appeared on the PR as a comment.

@andekande andekande force-pushed the fetch-all-comments branch from 38576d5 to 72e0295 Compare June 20, 2024 15:48
@shine2lay shine2lay merged commit c1751e2 into mergeability:master Jun 20, 2024
2 checks passed
Copy link

🎉 This PR is included in version 2.19.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@andekande andekande deleted the fetch-all-comments branch June 20, 2024 22:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants