An interesting article at http://eatcodeplay.com/why-we-killed-off-code-reviews/
I wasn't familiar with the 200 lines of code per hour study they cite from the IEEE.: http://www.pitt.edu/~ckemerer/PSP_Data.pdf.
That sounds excruciatingly slow, although not out of reason. There have been days in my history where I wrote thousands of lines of code. Sitting down for 8-10 hours of line-by-line would have been excruciating (and some of that code was still running as of this year, so I know it survived my developer demise). It's interesting to see the issue of context switching infesting everything, even what's considered a best practice.
"You push a pull request. Someone else has to stop what they’re doing to review it. Meanwhile, you’ve started the next feature. If the reviewer has questions, you’re pulled back into that conversation with possible changes to make after the fact."
No comments:
Post a Comment