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

This ignores a bunch of higher level and long-term concepts like maintenance, complexity and extensibility.

With only "it works" you end up like this (iconic HN comment on Oracle codebase): https://news.ycombinator.com/item?id=18442637





Yes, my point is that you don't even have "it compiles" as a way to measure a code review. Maybe you did a great job, maybe you did a terrible job, how do you tell?



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

Search: