EmbarkStudios / octobors

Rust program for automerging PRs based on a few rules
Apache License 2.0
40 stars 2 forks source link

Support asking the bot why a PR isn't merged #32

Closed davidpdrsn closed 1 year ago

davidpdrsn commented 1 year ago

If you're running octobors with comment_requests_change = true then you sometimes run into scenarios where everyone has approved a PR, all checks passes, but someone has left a drive by comment. In that case the PR wont be merged because the comment is counted as a request-for-change. If people aren't aware of this it appears as if the bot is broken and just refuses to merge their PR, which is annoying.

One way to improve this would be to support asking the bot why a PR isn't merged perhaps by commenting on the PR with something like @octobors y u blocked. The bot would then reply with which checks haven't passed which should make it clear that comments counts as requests-for-change, or perhaps that CI failed.