[Libreoffice] Reviewing rules

Kohei Yoshida kyoshida at novell.com
Fri May 20 09:20:49 PDT 2011


On Fri, 2011-05-20 at 10:45 +0200, Jan Holesovsky wrote:
> Hi all,
> 
> I've seen some patches that were reviewed, but not pushed into the -3-4
> branch (!!!), so just to remind you:
> 
> - the first reviewer pushes to libreoffice-3-4 (commits/cherry-picks
>   with -s to get the Signed-off-by: message automatically)
> 
> - the third reviewer cherry-picks the commit(s) from libreoffice-3-4 to
>   libreoffice-3-4-0, again with -s

Also, I'd like to have some consistency in the [REVIEW] tag.  We have so
far

[REVIEW]
[REVIEWED]
[REVIEWED - 1 more needed]
[REVIEW REQUEST]
etc.

and I'm having hard time coming up with a good filter rule in Evolution
to include all of these.  Evolution doesn't support regexp search which
also adds to the difficulty.

How are others coping with this?

Kohei

-- 
Kohei Yoshida, LibreOffice hacker, Calc
<kyoshida at novell.com>



More information about the LibreOffice mailing list