At that point then disable merge checks and let them merge without a review. If there is a problem it's on them
replyThis is my current strategy, it's working great. Half the team has been fired for slop and the other half got fired for not doing anything.
replyI'm sure this person's manager knows that having trouble getting PRs reviewed can (but not always) be a signal of a deeper problem. It could be that no one one the team knows the domain, it could be that no one like the person, but most likely it's that the PRs are frequently bad and no one wants to bother.
replyOr, I might say, why review the PR. Get Claude to do it? Why do I need to spend my time and attention and this person does not?
reply