perlquestion
radiantmatrix
<p>I think of myself as a moderately good developer. I have a reasonable
amount of experience, I know about common patterns, I reuse code when possible,
I try to follow good development practice.
<p>I use test-driven development, I religiously use a revision control systsem.
I use [cpan://Devel::Cover] to check that my tests cover 100% of the code I
write. Likewise, I ensure that I have good [cpan://Pod::Coverage].
<p>I've done informal code reviews before -- at many of my clients, my
co-workers disdained code review, so I found ways to do it "on the sly": "Hey,
I'm not sure about this module, would you look at it with me?"
<p>Now, though, I have an opportunity to do more structured and formal code
review. I don't want to bury the process in formality and paperwork, but it is
important that I can point a manager at documentation and say "yep, I
<em>really</em> did code reviews, here's the proof".
<p>Can the members of the Monestary suggest:<ol>
<li>Ground rules for having useful code reviews
<li>Ideas for what kinds of documentation I should have to prove they've been
done
<li>Any other information about conducting semi-formal code reviews in a way
that's really useful for developing (esp. in Perl).
</ol>
<p>I'd certainly benefit from the experience of my esteemed brothers and sisters.
<!--
<p><small><b>Updates:</b><ul type='square'>
<li>date : desc</li>
</ul></small></p>
-->
<div class="pmsig"><div class="pmsig-375088">
<small>
<small><font color='#000000'><</font><font color='#1a1a1a'>–</font><font color='#343434'>r</font><font color='#4e4e4e'>a</font><font color='#686868'>d</font><font color='#828282'>i</font><font color='#9c9c9c'>a</font><font color='#b6b6b6'>n</font><font color='#d0d0d0'>t</font><font color='#eaeaea'>.</font><font color='#d0d0d0'>m</font><font color='#b6b6b6'>a</font><font color='#9c9c9c'>t</font><font color='#828282'>r</font><font color='#686868'>i</font><font color='#4e4e4e'>x</font><font color='#343434'>–</font><font color='#1a1a1a'>></font></small>
<!--<-</small><b>radiant</b>.<b>matrix</b><small>-></small-->
<br><a href='http://radiantmatrix.org/'>Ramblings and references</a>
<br>“A positive attitude may not solve all your problems, but it will annoy enough people to make it worth the effort.” — <em>Herm Albright</em>
<br><em>I haven't found a problem yet that can't be solved by a well-placed [http://en.wikipedia.org/wiki/Trebuchet|trebuchet]</em>
</small>
</div></div>