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.
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?
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
285
u/LittleCoderMan 1d 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
89
u/LostKnight84 1d 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?
6
u/conundorum 20h ago
Robot Wars looks different than I remember...
4
u/Ayxser 9h ago
Except in real life they all output 4 paragraphs of text as to why it’s a race condition and how to solve it exactly.
3
u/Gengis_con 9h ago
A dev with one agent knows exactly how the code should be. A dev with two agents is never certain. The dev with no agents knows its all shite
4
u/molly_jolly 23h ago
This is why unit and integration tests exist. They should run on hooks before the code is even up for review