3 ms·
> If something "seems right/passed review/fell apart" then that's the reviewer's fault right? No, it's the author's fault. The point of a code review is not to
by prewett 11mo ago
> If something "seems right/passed review/fell apart" then that's the reviewer's fault right?
No, it's the author's fault. The point of a code review is not to ensure correctness, it is to improve code quality (correctness, maintainability, style consistency, reuse of existing functions, knowledge transfer, etc).
- spopejoy 11mo agoI mean, that's just not true when you're talking about varying levels of experience. Review is _very_ important with juniors, obviously. If you as sr eng let a junior put code in the codebase that messes up later, you share that blame for sure.