Mark Dain On Thursday I called out a co-worker on some bad code he'd written on the front end and that caused some tension in the office. What is the best approach to letting people know they need to write more maintainable code and stick to accepted conventions (everyone else uses tabs, he uses spaces). Stuff like that, it's small but it really drags down the quality which we're trying to raise.

Reverendshaft In our development department, we have an open peer review culture, where other developers are /expected/ to raise concerns over the code other developers present. It's not uncommon for an intern to question a lead developer's work, and it raises the bar without putting any one person under an uncommon level of scrutiny.

Mark Dain That's a really nice approach. I wish we had that at my office. We're working towards implementing version control so I'm hoping peer review will become more common.