Comment Comment by monkeyelite > This is the same point three timesNo it isn’t. Fake work, synchronization, and context switching are all separate problems.> code reviews are a way to catch thoseI said you can do reviews - but there is no reason to stop work to do them.Why not require two or three reviews if they are so helpful at finding mistakes?I agree everyone makes mistakes - that’s why I would design a process around fixing mistakes, not screening for perfection.How many times have you gone back to address review comments and introduced a regression because you no longer have the context in your head? parent monkeyelite 1 year ago
No it isn’t. Fake work, synchronization, and context switching are all separate problems.
> code reviews are a way to catch those
I said you can do reviews - but there is no reason to stop work to do them.
Why not require two or three reviews if they are so helpful at finding mistakes?
I agree everyone makes mistakes - that’s why I would design a process around fixing mistakes, not screening for perfection.
How many times have you gone back to address review comments and introduced a regression because you no longer have the context in your head?