2026-10-26 06:06 UTC
Oh it’s already easy. You start asking the submitter to explain the code. Sometimes it’s just visually obvious, too.
I’ve always had the opinion of it not being slop if you:
Take ownership and fully understand the code
Make it look like a human wrote it, in other words, scrutinize, re-factor and re-format the code for legibility, performance and maintainability before submitting your patch
This effectively forces you to do research to properly understand how your patch is implemented and puts you through the mental exercise of cleaning up the code. It also more often than not mostly replaces the code the LLM generated anyway, making it yours.
I say all this as someone who turns around a lot of PRs at work for being slop. Come back to me when you can take ownership of your git blame.
Replies (1)
-
@MrScottyTay@sh.itjust.works 2026-10-26 18:35
No matter how good an ai can be, what it writes should never be pushed as-is. It’s just like you shouldn’t ever have copy-pasted snippets from stack overflow to fix bugs.