<tt><font size=2>Rick van Rein <rick@openfortress.nl> wrote on 23/10/2009
11:36:31:<br>
<br>
> Stephen,<br>
> <br>
> You are very clear in your request for documentation backing having
done<br>
> code reviews and tests.  You summed up lists of items you would
like<br>
> to see in them.  Clearly, you have a better background in ISO-9000<br>
> than most of us (at least more than I).<br>
> <br>
> With this in mind, is it perhaps an idea that you create a template
for<br>
> such reports, so testers/reviewers only need to fill it out?  It'll
help<br>
> searching through them at a later time for the people who are keen
on<br>
> ISO-9000 issues and it'll help to ensure that the reports do not miss<br>
> out on information that is easily overlooked otherwise.<br>
> <br>
> <br>
> Cheers,<br>
>  -Rick<br>
</font></tt>
<br><tt><font size=2>I could do a template, but I don't want to be too
prescriptive.  Providing each report contains the following information:</font></tt>
<br>
<br><tt><font size=2>Files checked and their revision</font></tt>
<br><tt><font size=2>Date of review</font></tt>
<br><tt><font size=2>Reviewer</font></tt>
<br><tt><font size=2>Comments on the files</font></tt>
<br>
<br><tt><font size=2>... that should be enough.</font></tt>
<br>
<br><tt><font size=2>If the review raises issues that are sufficient to
warrant a change, a story should be raised in Pivotal Tracker and the ID
of the story added to the review.</font></tt>
<br>
<br><tt><font size=2>Reviews should be passed to Rickard, who should file
them. (If we decide its worth it, at some time we could publish them on
the wiki)</font></tt>
<br>
<br>
<br>
<br><tt><font size=2>Stephen</font></tt>