3 ms·
We're very small (3 full time devs, a mostly full time tester, a part time student, and another student temporarily full time), and the three full timers are pr
by PeterWhittaker 5y ago
We're very small (3 full time devs, a mostly full time tester, a part time student, and another student temporarily full time), and the three full timers are pretty seasoned, so this may not apply universally, but....
We don't review code as a matter of course. But if anyone of us writes code that doesn't pass our personal smell test, or that we think it won't pass someone else's smell test, or <insert criteria here>, screens are being shared and ideas tossed about.
We almost always <solve the problem we were having|create a better solution|decide to accept the debt and open an issue until we have a better way>, in pretty short order.
So it's sort of JIT/OnDemand code reviews. Kinda.