Comment It's worth it (Score 0) 345
Email pass-around Source code management system emails code to reviewers automatically after checkin is made.
It works much better than expected. We started on a 1st version about 2 years ago with 4 developers. After the shipment, we had a "what was good/bad, what could be better"-meeting. The bad things were the same shit as in so many projects: not documented dirty and unmaintainable code, which looked like a quick perl 1liner to check an e-mail address, but it was Java and C++! If your co-worker is touching the code parts you're interesed in or working on, you have to review his check ins and vice versa. At least it ensures, that the checked code is:
a) Documented
b) Doesn't contain 100+ characters long undocumented regex
c) Doesn't look like your favorite pasta dish
d) You know, what is going on and it makes it easier to maintain that code in the feature
By the time we doubled the team and it seems to help. If some developer gets 2-3 "WTF?!!"-mails a day from his co-workers, he starts to think before commiting the next time..