3 ms·

> We review the code all together (max 2 ppl) just before the release and that's it. Is there a reason why you don't review the code immediately? You could als

by lapcat 2y ago

> We review the code all together (max 2 ppl) just before the release and that's it. Is there a reason why you don't review the code immediately? You could also do that without PRs.

pavel_lishin 2y ago
Yeah, I've heard arguments for this, and it always feels weird to me. Why go so long without review? That's just giving developers - junior and senior - opportunity to dig themselves into a hole that they don't notice until someone points out, "hey, your last five commits are buggy, but that matters less than the fact that they're implementing the wrong thing."
- zie1ony 2y ago
  Good point, but my team is small and has only top engineers with both soft and hard skills. Many times you can do much more with team of 3 than team of 10 ppl.
  - Aeolun 2y ago
    If the three sit next to one another I agree that PR’s are overkill. If they’re remote it becomes harder I think.