3 ms·
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
by cmpb 11y ago
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.
- Kaedon 11y agoSmartBear 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-practices-of-peer-code-review.pdf http://smartbear.com/smartbear/media/pdfs/wp-cc-11-best-prac...