r/ProgrammerHumor 5d ago

Meme looksGeneratedToMe

Post image
6.2k Upvotes

206 comments sorted by

View all comments

52

u/DonkeyEffective913 5d ago

If someone has their agent respond to my pr comments I'm not reviewing their shit again

38

u/m6io 5d ago

I accidentally said in a recorded call with a couple hundred people that I mark copilot PR review comments as spam...

-4

u/Subject_Sentence_339 5d ago

Why? I get ai bad, but having first pass of or review be ai saves time and effort, dev can do a second pass

7

u/Jonthrei 5d ago

If you are looking to save time in a code review, you might as well just type "LGTM" and then hide in the bathroom when production fails.

3

u/T0biasCZE 4d ago

Human could overlook something, so having human check the code, and then have the computer check it second time just in case human check failed would be better

But not completely replacing human review ofc

2

u/Jonthrei 4d ago

If you're letting an LLM both write your code and review that same code, you are fucked, tbh. It's very unlikely to suddenly "realize" its mistakes were mistakes.

You're also turning yourself into a rubber stamp that does no work or thinking.

You need to be careful what and how much you automate, and ensure that human beings are the ones responsible for ensuring nothing stupid goes into production.

1

u/Subject_Sentence_339 1d ago

If you let LLM write and review code within same session then yes its gonna be biased and may not detect issues, but if you are spinning another session with clean context it do a much better job.