Things I think about during a code review: Is the intent clear
Things I think about during a code review:
Is the intent clear?
Is there needless duplication?
Could existing code have solved this?
Is this an atomic commit?
Could this be simpler?
Are there logic issues?
Does this follow our standards?
Are the tests comprehensive and clear?