I am making two proposals that will help &quot;tighten&quot; up our QA procedures a bit in order to facilitate clarity and transparency in our patch submission/acceptance workflow. Currently the workflow is described in the wiki here:<br>

<br><a href="http://wiki.koha-community.org/wiki/Bug-enhancement-patch_Workflow#Steps">http://wiki.koha-community.org/wiki/Bug-enhancement-patch_Workflow#Steps</a><br><br>1. I propose that we modify step 5 to read:<br><br>

&quot;The patch is checked and signed-off by the QA team member. Then the bug status is set to Passed QA&quot;<div><br></div><div>This will ensure that we have clarity that the patch was, indeed, touched by a member of the QA team, as well as increasing the accuracy of QA stats in git.</div>

<div><br></div><div>2. I propose that the RM be the QA of last resort. At present the stats show that the RM is doing the majority of the QA&#39;ing. &quot;Last resort&quot; is a condition evoked by all members of the current QA team acknowledging that no one among them has the time, etc. to do QA on a particular patch the RM feels needs to be pushed OR by a bug remaining in the &quot;Signed Off&quot; status beyond a fixed time period of four weeks. This mechanism would address concerns of bug stagnation by allowing ample time for QA members to at least glance at a bug and determine of they do or do not have time to give to that bug as well as having a catch mechanism for bugs falling through w/o notice.</div>

<div><br></div><div>Here are some statistics on sign-offs by the currently listed QA team for commits between the 3.6.0 and 3.8.0 tags:</div><div><br></div><div>NOTE: 100% of our patches have the minimum required 1 sign-off by any community member</div>

<div><br></div><div>Total commits: 1086</div><div><br></div><div>Sign-offs by QA Team members:</div><div><br></div><div>Marcel de Rooy - 43</div><div>Jonathan Druart - 5</div><div>Paul Poulain - 538</div><div>Ian Walls - 25</div>

<div><br></div><div>Total commits w/sign-off by a QA team member: 611 OR 56.3%</div><div>Total commits w/sign-off by RM: 538 OR 49.5%</div><div>Total commits w/sign-off by non-RM QA member: 73 OR 6.7%</div><div><br></div>

<div>These figures reasonably agree with the stats posted here: <a href="http://blog.bigballofwax.co.nz/2012/04/24/statistics-for-koha-3-8-0/">http://blog.bigballofwax.co.nz/2012/04/24/statistics-for-koha-3-8-0/</a></div>

<div><br></div><div>You may reproduce them on your local git repo master branch by using variations of the following:</div><div><br></div><div>git log --grep=&quot;^Signed-off-by: &lt;regexp&gt;&quot; --no-merges --oneline 52c666edce42..5c32a9f811d | wc -l</div>

<div><br></div><div>Lets have some discussion on the two proposals above, and I&#39;ll put them on the agenda for the next IRC meeting to be voted on.</div><div><br></div><div>Kind Regards,</div><div>Chris</div>