Hacker Newsnew | past | comments | ask | show | jobs | submitlogin

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


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.




Guidelines | FAQ | Lists | API | Security | Legal | Apply to YC | Contact

Search: