A more hands on review process

Aaron J. Seigo aseigo at kde.org
Mon Aug 4 23:36:16 BST 2008


On Monday 04 August 2008, Stephen Kelly wrote:
> Aaron J. Seigo wrote:
> > On Thursday 31 July 2008, Stephen Kelly wrote:
> >> I propose a review process based on review criteria instead of time.
> >> Reviewers would 'tick the boxes' to confirm that they have reviewed it,
> >> and what they looked for. Multiple reviewers could tick only some of the
> >> boxes each if necessary (I'd have no idea if something works on solaris
> >> or has security flaws for example), and together create a complete
> >> review.
> >
> > would this take the form of a web-based collaboration app? or?
>
> I was thinking email on k-c-d. If there's a suitable web-app for this kind
> of thing which actually makes it easier, then that's cool too. 

well, my experience in encouraging peer review within plasma is that mailing 
lists simply don't cut it. they are a perfect aid, but the review needs to be 
handled elsewhere.

the reason is that mailing lists archives are too difficult to track through, 
have no easy way to mark as "done!", have no work flow associated with them at 
all, really. it's a conversation medium, not a workflow medium

a webapp really helps there (even if the webapp ultimately CC's a mailing list 
=)

> Is
> review-board suitable for applications, or mostly just patches (I have no
> idea).

it's not really suited for this at all, no.

-- 
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43

KDE core developer sponsored by Trolltech

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20080804/eaa0b3c4/attachment.sig>


More information about the kde-core-devel mailing list