“I don’t understand this” reflects a valid feeling but is not a constructive comment in a code review. What about using “Would you explain this to me?”
A good code review is not a series of walls or hoops for the writer to jump through. It’s a critical discussion conducted by a team. Code reviewers should be collaborators not obstacles.
Thanks for your comment. I think the unfinished code might be a symptom of a lack of direction rather than a need to move fast. There are large organization changes happening and the chaos is making it hard to see what the next steps are.
I’ve tried a couple of these things so far. I moved teams to try and cut out some infrastructure improvements but got foiled when the other side of the company decided unilaterally to change cloud hosting providers (My boss left the company after that).
A good code review is not a series of walls or hoops for the writer to jump through. It’s a critical discussion conducted by a team. Code reviewers should be collaborators not obstacles.