Comment on The "I don't know, Claude wrote this" pandemicComments−newsomix9xl16dI 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?" EtcI've had code approvers who gave me tons of crap, even over smaller edits. You too can be that kind of reviewer!−rags2riches15dMy favorite reviewers are those that say "I'm not sure I see the whole picture, can you walk me through this?"−nasretdinov15dYeah 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.−DarmokTanagra15dDo 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.−newsomix9xl15dYes, 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).
Comments
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!
My favorite reviewers are those that say "I'm not sure I see the whole picture, can you walk me through this?"
Yeah 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.
Do 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.
Yes, 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).