[SVN] Peer Code Review
I remember sitting in on code reviews early in my career. You took your code, put it up on a projector, and the architects—shadowy figures in the back of the room—ripped it to shreds. These marathon sessions would take all day and were completely exhausting. They certainly improved the code, but they wreaked havoc with both release schedules and people’s psyches! Nowadays, I do code reviews a little differently. Instead of submitting our code to the senior architects, we do peer code reviews. Here’s how it works: I go away and write some code, including tests for that code. I run all the tests and tweak until I’m happy with the work. It functions, it’s structured properly, it conforms to all of our style requirements, and it’s got a really clever solution to that sorting problem. I commit the code to my feature or developer branch. It’s not in mainline code yet, and won’t ship (this is important!). I ask for review. If we’re working in Git, then I do it by creating ...