<p dir="ltr">String changes can be calculated algorithmically. I suspect that the other complexities are largely a function of lines of code, number of files touched, and number of dependencies... but I'm sure that there other factors. The smaller the patch, the more likely that the complexity could be determined by a machine. </p>
<div class="gmail_quote">On Aug 31, 2015 1:41 AM, "Katrin Fischer" <<a href="mailto:Katrin.Fischer.83@web.de">Katrin.Fischer.83@web.de</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I'd be ok testing Jonathan's suggestion of moving to Passed QA directly<br>
for for bugs with'string change' or 'trivial' complexity.<br>
QA and RM could still reset to 'Needs signoff' for a second opinion when<br>
they feel it's needed.<br>
<br>
I think the patch complexity can only be set after developing a patch -<br>
so making it mandatory might not work as a technical solution here.<br>
Maybe we can encourage the use of the field a bit more?<br>
<br>
Currently for most of the patches in the 'Needs Sign-off' queue the<br>
patch complexity is not set - so it's hard to tell how many patches this<br>
change would affect.<br>
<br>
Katrin<br>
<br>
Am 25.08.2015 um 14:57 schrieb Fridolin SOMERS:<br>
> +1<br>
><br>
> On those 200, how many have a severity of trivial or lower ?<br>
><br>
> Le 20/08/2015 15:51, Jonathan Druart a écrit :<br>
>> Hello devs,<br>
>><br>
>> I would like to suggest a simplification of the integration workflow<br>
>> for some patches.<br>
>> Indeed the signoff queue is back to a critical threshold (200) and the<br>
>> signoffer's activity is very low.<br>
>><br>
>> Being part of the QA team, I try not to test patches in the Needs<br>
>> Signoff queue as I loose my QA token.<br>
>> The active members of the QA team are very limited and when we do SO,<br>
>> this increase the QA queue.<br>
>> As a result there is less chances my patches are QAed as there are<br>
>> more and more patches in the QA queue. It's a bit egoistic, but we all<br>
>> have the same problem.<br>
>><br>
>> So I suggest that for some patches, the ones with a minor severity,<br>
>> small or template changes could, IMO, bypass the signoff step.<br>
>><br>
>> I would like to discuss this during the next dev meetings, but I am<br>
>> keen to get some feedbacks on this idea.<br>
>><br>
>> Cheers,<br>
>> Jonathan<br>
>><br>
>> PS: If I am correct, we already have discussed about that previously.<br>
>> _______________________________________________<br>
>> Koha-devel mailing list<br>
>> <a href="mailto:Koha-devel@lists.koha-community.org">Koha-devel@lists.koha-community.org</a><br>
>> <a href="http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-devel" rel="noreferrer" target="_blank">http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-devel</a><br>
>> website : <a href="http://www.koha-community.org/" rel="noreferrer" target="_blank">http://www.koha-community.org/</a><br>
>> git : <a href="http://git.koha-community.org/" rel="noreferrer" target="_blank">http://git.koha-community.org/</a><br>
>> bugs : <a href="http://bugs.koha-community.org/" rel="noreferrer" target="_blank">http://bugs.koha-community.org/</a><br>
>><br>
><br>
<br>
_______________________________________________<br>
Koha-devel mailing list<br>
<a href="mailto:Koha-devel@lists.koha-community.org">Koha-devel@lists.koha-community.org</a><br>
<a href="http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-devel" rel="noreferrer" target="_blank">http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-devel</a><br>
website : <a href="http://www.koha-community.org/" rel="noreferrer" target="_blank">http://www.koha-community.org/</a><br>
git : <a href="http://git.koha-community.org/" rel="noreferrer" target="_blank">http://git.koha-community.org/</a><br>
bugs : <a href="http://bugs.koha-community.org/" rel="noreferrer" target="_blank">http://bugs.koha-community.org/</a><br>
</blockquote></div>