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

If you give up on unit tests and code review then the code is "yours" instead of "ours" and your coworkers will not want to collaborate on it with you.

However, this has to be substantive code review by technical peers who actually care.

Unit tests also need the be valued as integral to the implementation task. The author writes the unit tests. It helps to guide the thought process. You should not offload unit tests to an intern as "scutwork".

If your code is sloppy, a stylistic mess, and unreviewed, then I am going to put it behind an interface as best I can, refer to it as "legacy", rely on you for bugfixes (I'm not touching that stinking pile), and will probably try to rally people behind a replacement.



In my experience that did not happen. I've been lucky perhaps to always work with engineers I trusted.

And frankly, giving ownership to code ("it's yours") has, also in my experience, been an excellent way to give an engineer "pride of ownership". No one wants to have that "stinking pile".




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

Search: