Date: prev next · Thread: first prev next last
2012 Archives by date, by thread · List index


On Mon, Aug 20, 2012 at 03:39:29PM +0200, Jan Holesovsky wrote:
But this all is the implicit meaning of the +2, isn't it?

No, code-review means: the _code_ looks good to my human developer eyes.
verified means: builds, and passes all automatic or manual tests I care about.

Tinderboxes will set the verified flag only for example. A non-developer
running a build with the fix and checking for the bug can also set verified --
although he cannot really judge the implementation.

The separation makes some sense. For example we want tinderboxes to build as
much as possible. OTOH we dont want tinderboxes to build a change that some
anonyous guy uploaded without one of our commiters at least having a look first.(*)

Also ideally in the end you will set the change to +2 codereview and a
tinderbox later moves along and builds/tests the change. Once the tinderbox
reports it is happy too, the change gets merged.

Best,

Bjoern

(*) Nifty little keylogger implementation you injected in gbuild there ...

Context


Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.