3 ms·
I have worked with quite a few people committing code they didn't fully understand. I don't meant this as a drive by bazinga either, the practice of copying co
by _heimdall 7mo ago
I have worked with quite a few people committing code they didn't fully understand.
I don't meant this as a drive by bazinga either, the practice of copying code or thinking you understand it when you don't is nothing new
- allajfjwbwkwja 7mo agoPre-LLM, it was much easier for reviewers to discern that. Now, the AI-generated code can look like it was well thought out by somebody competent, when it wasn't.
- jhide 7mo agoHave you ever reviewed an AI-generated commit from someone with insufficient competence that was more compelling than their work would be if it was done unassisted? In my experience it’s exactly the opposite. AI-generation aggravates existing blindspots. This is because, excluding malicious incompetence, devs will generally try to understand what they’re doing if they’re doing it without AI
- allajfjwbwkwja 7mo agoI have. It's always more compelling in a web diff. These guys are the first coworkers for which it became absolutely necessary for me to review their work by pulling down all their code and inspecting every line myself in the context of the full codebase.
- bandrami 7mo agoI think the issue is not that the patches are more compelling but that they're significantly larger and more frequent
- abustamam 7mo agoI try to understand what the llm is doing when it generates code. I understand that I'm still responsible for the code I commit even if it's llm generated so I may as well own it.
- enneff 7mo agoYes and if they copy and paste code they don’t understand then they should disclose that in the commit message too!