3 ms·
I hope I don't sound too argumentative but, no, I mean absolutely required. I review my own code (and not just casually, either) a few days later and where I wo
by thenduks 17y ago
I hope I don't sound too argumentative but, no, I mean absolutely required. I review my own code (and not just casually, either) a few days later and where I work we all read every commit.
- tptacek 17y agoIt's so "absolutely required" that most of the most popular web applications, all written in dynamic languages, are not the product of reviewed codebases. You don't sound argumentative. You sound hyperbolic.
- thenduks 17y agoI don't think we're on the same page about what 'code review' is. I'm only talking about having your co-workers/peers look at your code and keep up to speed on what's happening to the code base you're all working on. Do you have a citation for 'most' of the popular web applications not being code reviewed? Have the developers at, say, twitter said they ignore commits and code written by co-workers?