4 ms·
That's a hard perspective for many developers to adopt because people understandably prioritize their own tasks and deadlines. I think it needs manager support
by cpeterso 4y ago
That's a hard perspective for many developers to adopt because people understandably prioritize their own tasks and deadlines. I think it needs manager support and even encouragement because it's a global optimization for your team's productivity. Reviewing and merging pending PRs will deliver value to your product sooner than procrastinating on pending PRs so you can write new code that will then need to wait for some other developer to review it. It can be a vicious cycle.
- salamanderman 4y agoAdditionally, at least in small-medium sized teams, I've seen teams become over reliant on the small number of people who prioritize and respond quickly to code review requests. A benefit of not prioritizing code review as an individual is that it can naturally spread the review responsibility, and understanding of the code, more broadly.