Hacker News new | ask | show | jobs
by AdieuToLogic 314 days 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.
1 comments

Of 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.