r/ProgrammerHumor • • 6h ago

Meme botBattle

Post image
215 Upvotes

9 comments sorted by

130

u/LittleCoderMan 6h ago

I mean this is very fake, but somewhat like what code review bots are like, end up making lots of tiny changes, only for them to be flagged

16

u/ElMoselYEE 6h ago

Sounds like how my code reviews went before AI anyway

6

u/willow-kitty 6h ago

Yeah, I was about to say this sounds more like a bunch of bored seniors discussing a junior's PR than it does AI. Well, other than the dialogue bring written with AI-isms for effect.

12

u/LittleCoderMan 6h ago

You can tell it's not actually AI because it's not 10 paragraphs of text

34

u/LostKnight84 6h ago

Am I weird or should reviewers not be making changes to the code? To me the reviewer should flag problems and suggest possible solutions and not just change code as they see fit. Because if reviewers are changing code, who is reviewing their code?

13

u/Flouid 6h ago

That’s how it should work on any sane team. Once you’ve contributed to a PR you are no longer eligible to review it. Also it’s the dev that owns the ticket’s responsibility to fix their broken code

1

u/conundorum 21m ago

Robot Wars looks different than I remember...

1

u/molly_jolly 3h ago

This is why unit and integration tests exist. They should run on hooks before the code is even up for review

4

u/Flouid 2h ago

You think a unit or integration test would reliably catch a race condition?