You must log in or register to comment.
The worst case is when someone requires changes, you address them, but then they disappear/go on a leave.
If the repository rules require all conversations to be resolved before merging and only the original reviewer can mark them as solved, the PR is stuck forever even if the rest of the team approves it.
That sounds like something you bring up in the retro
This always happens to me when I’ve written some genius code. Takes so long to review it, because my caveman colleagues don’t understand it.
Yeah, this is my colleagues waiting for me, poor bastards
A 15000 line PR landing on a Friday evening for the lucky random reviewer to open on Monday. “Please approve it fast so we avoid too many conflicts.”