loading...

Discussion on: How to make code reviews developer-conflict free?

Collapse
danspratling profile image
Dan Spratling

That depends on your goals but if you feel like you're a blocker then:

Is it readable?
Does it do what it set out to do?

These are the the only things which really matter when reviewing code.

Enforcing biases only slow down the process.

If you have code standards which aren't being followed, question why that is. You may be able to automate that process.

If you find that PRs are overwhelming, try breaking them down into smaller more achievable tasks. This will help dev and review.