Hacker Newsnew | past | comments | ask | show | jobs | submitlogin

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.



Using a linter goes a long way for this. Maybe make linking part of the check in process.




Guidelines | FAQ | Lists | API | Security | Legal | Apply to YC | Contact

Search: