4 ms·
We have a simple rule: You commit it, you own it. If you vibe coded it that's okay. If it's garbage that's on you. Blaming the LLM doesn't count. So of course y
by BadBadJellyBean 1y ago
We have a simple rule: You commit it, you own it. If you vibe coded it that's okay. If it's garbage that's on you. Blaming the LLM doesn't count. So of course you have to read the code. You have to read it and understand it. Works well for us.
- yosito 1y agoAnother related rule: never commit code to the main branch that hasn't been read by at least two humans.
- AdieuToLogic 1y ago> Another related rule: never commit code to the main branch that hasn't been read by at least two humans. Passive code reviews ("read by at least two humans") are fraught with error. IMHO, a better mantra is: Never allow a merge to the main branch that does not have some amount of documented test coverage, be it unit, functional, and/or integration specs.
- BadBadJellyBean 1y agoOf course that is true but with AI you can add documentation and tests. You can have absolut garbage with all green tests. We do human reviews on PRs but in the end it's your code that you have to stand behind.