4 ms·
I always use AI to understand big diffs. And you can always reject a PR, bro with "I don't understand it to approve it. Maybe make it smaller or documents it b
by newsomix9xl 2mo ago
I always use AI to understand big diffs.
And you can always reject a PR, bro with "I don't understand it to approve it. Maybe make it smaller or documents it better?" Etc
I've had code approvers who gave me tons of crap, even over smaller edits. You too can be that kind of reviewer!
- nasretdinov 2mo agoYeah that's quite odd that people somehow now think you can't reject a PR because you can't understand it -- but initially that was one of the reasons to have code review in the first place.
- DarmokTanagra 2mo agoDo it enough and you become the problem. I ran into this at my previous company and ended up getting scolded for being a bottleneck when I was just sending back AI slop code.
- newsomix9xl 2mo agoYes, I couldn't complain at one place as the code Nazi reviewer was my manager, and one of the best I've had (managers not code Nazi reviewers).
- rags2riches 2mo agoMy favorite reviewers are those that say "I'm not sure I see the whole picture, can you walk me through this?"