Why Don't merge will be ignored in review lists?

[kachick]

In my understanding, WIP does not require coworker reviews. But Don't merge does not mean it. Don't merge finished the coding and needs reviews, but can not be merged from any reasons. How do you think it?

(If you have same thought with me, Other PRs of WIP should be ignored with another way? label or anything... 🤔 )

[ohbarye]

But Don't merge does not mean it. Don't merge finished the coding and needs reviews, but can not be merged from any reasons. How do you think it?

I believe that's totally depends on a team. It's simply that Don't merge doesn't require review in our team so far.

Then if I propose a feature to ignore PRs by other way (e.g. to specify words to be ignored), would your problem be resolved? 🤔

@review-bot ls author:kachick -ignore-words:"WIP","RFC"

[kachick]

Sounds 😍

[ohbarye]

Thanks for you opinion, give me some time. 🙇