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

Nice list. This is more or less what I look for. It's nice to see your rules of thumb.

Anyone have any suggestions for time-estimating code review? That's the biggest issue we've faced trying to implement code review into our workflow.



SmartBear has their own set of best practices and they recommend about 300-400 loc per hour[1]. I think that's probably about right. It's something I think I've gotten faster at over time, partially because we're improving from previous reviews and partially because it's a skill that develops.

[1]: http://smartbear.com/smartbear/media/pdfs/wp-cc-11-best-prac...




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

Search: