|
|
|
|
|
by balabaster
4199 days ago
|
|
Perhaps I'm missing your meaning: "It's just that you can wean off senior developers quicker" Code reviews are just as necessary for senior devs, but it's more about helping everyone to understand what just happened and why: Here's the problem, here's what I did to resolve it, here are the pieces of the system it touches. That way at least one other developer is in the loop. Two heads are always better than one. Someone else might spot something you've missed... and worst case scenario, if I get hit by a bus tomorrow, someone else knows what I did and there's less of a chance that someone will have to spend a week wading through my code trying to reverse engineer what I just did. |
|
It would clearly be better to have the chance someone spends a week wading through your code than explaining all the changes you made to avoid that happening. You'll spend more than a week doing that.
I don't disagree with your other points, though. Two heads are better than one, and code reviews are important for a slew of reasons at all levels.