it’ll work better for everybody involved for a blocked person to stay away from reviewing PRs from the person who blocked them.
From my understanding, there’s no good way of quickly knowing this without visiting the profile of each individual. The example in question was SS2K just notifying the author that something occurred, and noticed that he couldn’t be quoted like others, most likely after rebasing the PR. Was it suprising for SS2K to find out in that manner? probably. Was it passive aggressive? sure.
Unless there’s direct messaging or other attacks that I’m not aware of. I don’t see anything that warranted the reaction from the initial review comment. I think pull requests should be neutral ground, code changes should stand on their own merit.
Would it be easier for certain people to avoid contributions by others, and mostly avoid possibilities of a conflict arising? sure. But this goes back to silo’ing and breaking means of communication.
I don’t want to be in a situation where my future reviews are met with, “your opinion isn’t welcome here, go elsewhere or else…”. Not to mention that these tensions seems to have persisted for months. If you think my commentary is bad, just say why what you’re proposing is preferred in the conversation, or the changes are out of scope. There’s no reason to escalate it to a personal issue and propagate the animosity in the future contributions. There’s a lack of communication from both sides, which is why we are here, and now it seems that we need to put in place some policies around social behaviors.