3 ms·
Because PRs with AI need to be reviewed with a lot more scrutiny, simply because AI is good at generating code that looks good, but isn't necessarily correct.
by OptionOfT 7mo ago
Because PRs with AI need to be reviewed with a lot more scrutiny, simply because AI is good at generating code that looks good, but isn't necessarily correct.
So now you're looking at a PR that at face value looks good, but doesn't reflect the author's skill and understanding of the subject.
Meaning now you shift more work to the owners of the codebase, as they have to go through those verifications steps.
- whattheheckheck 7mo agoJust deprioritize it and make the mr openers do more verification
- rendaw 7mo agoWhat sort of verification?
- whattheheckheck 7mo agoEverything that a maintainer would need to prove to themselves to merge it can be codified in a pipeline. Or some kind of protocol for building those things in the MR so that any new behavior explicitly demonstrates the new states and transitions. This is hard if the new MR introduces a completely different paradigm outside the mental model of the reviewer and maintainer. Might be better off completely forking the project and running it in parallel aka taking on the maintainer duties if they feel so inclined to completely change things
- deleted 7mo ago[deleted]
- th3tekllc 7mo agoThis makes 0 sense. It shouldn't matter if AI wrote the PR or a human.
- jkubicek 7mo agoIf a person writes 100 lines of code there’s a (valid) assumption that someone thinks this 100 lines of code is worth writing. With AI it takes no effort to write 10,000 loc. Asking someone else to figure out if that code is worth merging just offloads the effort to someone else who didn’t ask for it.
- array_key_first 7mo agoIt makes sense, because humans and AI write code different. Because humans aren't AI. The types of mistakes AI writes into code would never be done by a human.