EmbarkStudios / octobors

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

Add config option to consider that "comment" reviews request changes #27

Closed bnjbvr closed 2 years ago

bnjbvr commented 2 years ago

Plus a few cleanups.

This is an attempt to fix #11: there's an optional additional configuration option that makes it possible to consider a "comment" review as a "request-change" review.

Pros:

Cons:

repi commented 2 years ago
  • a drive-by review "comment" can be blocking a PR from getting merged. In this case, I would think that if someone randomly did a drive-by review of my code, it would be legitimate to ask them for formal review thereafter, and get their approval. So that doesn't seem too unreasonable, and we can try it out and see if it's too often a problem or not.

Think have run into this before but I don't remember the context/repo, the draw back of that was that compared to reviewers, you can't remove a user that added a comment that is blocking.

Still think this is a decent approach and tradeoff though to try it out