2 ms·
> Developers spend six hours per week reviewing. This is a bit too much It's extremely difficult to adjust the time spent on reviews. The options are unattract
by nitwit005 2y ago
> Developers spend six hours per week reviewing. This is a bit too much
It's extremely difficult to adjust the time spent on reviews. The options are unattractive. Do you start blindly accepting changes when you hit the limit, or just stop and not let people merge code?
- dakiol 2y agoWhy should one block merging code? The idea of someone spending hours working on some code changes being blocked by another individual that doesn’t allocate time for reviewing is alien to me. We are all professionals, let people merge their changes and make sure you find time to review them. If you can’t review them and they still need your approval, then approve. Unless you don’t trust your colleagues. If that’s the case, then code review is doomed anyway