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