4 ms·
This is a great list! I'd like to suggest one if you don't mind 7. Review your own code before handing it over. Time is wasted when the submitter left out obv
by http-teapot 8y ago
This is a great list!
I'd like to suggest one if you don't mind
7. Review your own code before handing it over. Time is wasted when the submitter left out obvious issues such as typos, missing comments, unnecessary complexity, debug code, etc.
Consider the reviewer's time precious because context switching is expensive and she/he might not get back to it as quick as you'd like to.
Implementing clear guidelines and linting would help.
- matt_the_bass 8y agoUsing a linter goes a long way for this. Maybe make linking part of the check in process.