<p>On the road so cannot give more of an in depth answer to some questions but just wanted to welcome you and add a +1 on the save button addition. If you would like to add that please go ahead. If no one responds once i'm more settled I'll send another mail. Welcome to the commit digest crew again :-) </p>

<div class="gmail_quote">On Nov 7, 2012 4:18 PM, "Jürgen Appel" <<a href="mailto:jeeemeeel@gmail.com">jeeemeeel@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
hello everyone, glad to be here!<br><br>Today I started my first round of reviews. I included most of the bug fix commits (those with a bko number) and also some which mentioned "this fixes..." but weren t related to a bug number.<br>

These commits alone amounted to more than the 5% rule would allow.<br><br>I suspect I m not  just having a bad day : /<br>(update: now that I read the 'classification guide'¹ (which I originally thought is only to be read by the classifiers) I feel even worse. Is the workflow of reviewer here that the reviewer adds the bugs to the classifiers table - who then does the triaging of the bugs, like checking the severity level?) <br>

<br>Secondly, rule no. 7 of the "Specific Guidelines"¹ shines at me with all its ambiguity and I fail to grasp it's meaning. Does it say that commits to feature branches (i.e. not being trunk) should exckuded - or that the commits which merge the feature branch into trunk should be ignored (i.e. anything saying 'merge')?<br>

In case it's the latter, will the commit message "merge: this finally brings my long awaited feature xy into trunk" make the commit interesting because it shows that 'xy' is included for the next feature release?<br>

<br>For some reason I d like to add to the guide¹ that after reviewing the commits, the reviewer is encouraged to hit the "save" button located in the lower right corner. I know it s not really a guideline, but it might help just in case "someone" fails to remark that button..<br>

<br>Now an easy one: do we use irc?<br><br>greetings, Juergen.<br><br><br>1: <a href="http://community.kde.org/Commit_Digest/Guidelines" target="_blank">http://community.kde.org/Commit_Digest/Guidelines</a><br><br>
<br>_______________________________________________<br>
Digest mailing list<br>
<a href="mailto:Digest@kde.org">Digest@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/digest" target="_blank">https://mail.kde.org/mailman/listinfo/digest</a><br>
<br></blockquote></div>