6 ms·
Reviewing someone else's large pull request is like having a second task in parallel with what you are working on yourself!
by 2sk21 1y ago
Reviewing someone else's large pull request is like having a second task in parallel with what you are working on yourself!
- chrisweekly 1y agoIt's not "like" another task, it IS another task!
- osigurdson 1y agoYeah but it is just a quick look, "yep", "yep", "oh what about this"?, "wow we dodged a bullet there". Its like self managed error correction that the collective does on its own. Fast, simple and produces good results. The less software you write the more this resonates.
- eterm 1y agoSo don't do it in parallel. Completely park other tasks, spend time on the review and record that time appropriately. There's nothing wrong with saying you spent the previous day doing a large review. It's a part of the job, it is "what you're working on".
- sexyman48 1y agoYou might as well go into HR. Everyone knows reviewing other people's PRs is like nurturing their kids at the expense of your own.
- eterm 1y agoI don't understand what you're trying to say.
- mystifyingpoi 1y agoWell, then just don't play the game. Make a decision in the team, that everyone accepts everyone's PR immediately without any review. At least you won't have to wait.
- sexyman48 1y agoI'm semi-retired now, but I spent most of my career at a Bell Labs-caliber place (I was the dumbest person there) before "PR" and "code review" became part of the lexicon, and yes, everyone was good enough not to mess things up too badly.