4 ms·
As a team. Have a working agreement. What does good code and code reviews look like? What does it mean for a task to be ready to be started and what is your tea
by sethammons 8y ago
As a team. Have a working agreement. What does good code and code reviews look like? What does it mean for a task to be ready to be started and what is your team's definition of done (level of testing, sign off of a code review, actually deployed, etc). These should be actual discussions and should probably be written down.
When it comes to a particular code review comment, either it is reasonable or unreasonable. If you disagree, say so with an argument. Referring to a working agreement or standard is helpful. Either that is enough or it is not. In the case that it is not and the issue continues to be unresolved, grab a more senior person (hopefully on the same team) to weigh in. Maybe the comment needs to slide for now, but become a ticket in the backlog. If somehow this is still not resolved, bring in your manager.