On Mon, 2016-07-04 at 18:08 -0500, Norbert Thiebaud wrote: > On Mon, Jul 4, 2016 at 3:04 PM, David Ostrovsky <d.ostrov...@gmx.de> > wrote:
[...] > Have you read the minutes you answered to ? > "if +2 __by author__ and +1 by jenkins -> auto-push" > > > > > project:core > > branch:master > > is:open > > is:mergeable > > label:Code-Review+2 > > label:Verified+1,c...@libreoffice.org > > NOT label:Code-Review-2 > > NOT label:Verified-1 > > that does not implement the rule desired. see above > we want Code-Review+2 _from the author_. > that query return 4 records instead of 2 (at the time of testing On second thought it would add some value to gerrit upstream to be able to detect change owner votes, e.g.: "label:CodeReview+2,change_owner". So, I uploaded a change to gerrit: [1] and teached GQL or more specifically label predicate from the secondary index to emit additional value per change owner vote on labels during reindex-phase and thus being able to detect those votes: [1]. With this patch applied there is native way with GQL to do what you want: project:core branch:master is:open is:mergeable label:Code-Review+2,change_owner label:Verified+1,c...@libreoffice.org NOT label:Code-Review-2 NOT label:Verified-1 I applied this commit on master (upcoming 2.13) as this is gerrit policy for adding new features, but it would be trivial to backport this non-intrusive diff to LO gerrit version (2.11.8), reindex, done. [1] https://gerrit-review.googlesource.com/79760 _______________________________________________ LibreOffice mailing list LibreOffice@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice
