3 ms·
That‘s the point. Everybody can, we just don‘t require it. Leaving it up to the individual developer to decide when it‘s right.
by thomaspaulmann 5y ago
That‘s the point. Everybody can, we just don‘t require it. Leaving it up to the individual developer to decide when it‘s right.
- Nursie 5y agoThen IMHO you're likely to get an odd culture of people requesting reviews or not based on perceived urgency, and also you're likely catering to those who consider it a point of pride not to need them, which is not a good attitude. We don't insist on review because we don't trust people, we insist on it because it catches problems, spreads knowledge around and helps developers.
- postalrat 5y agoIMO the biggest advantage for not requiring code reviews for every change is to concentrate code review efforts to where it's needed most: changes with higher risk. Sure, every change has some degree of risk. But obviously some changes have more and if you are treating all changes equally then you may overlook the important changes.
- Nursie 5y agoI'm not sure that requiring a review implies treating all changes equally, but it does at least get some eyes on everything, and IMHO that's important.
- schwartzworld 5y agoSo only the conscientious devs get code reviews. Great.