[darcs-devel] review team: please screen your own patches!

Ganesh Sittampalam ganesh at earth.li
Sat Aug 13 14:32:22 UTC 2011


Hi,

I'd like to remind the review team that they really should be pushing
their own patches to screened promptly, otherwise much of the point of
screened is lost.

Currently the following patches seem to be in this state:

http://bugs.darcs.net/patch629
http://bugs.darcs.net/patch630
http://bugs.darcs.net/patch631
http://bugs.darcs.net/patch632
http://bugs.darcs.net/patch635

If you're not sure it's ready, use "in-discussion". If you don't get any
comment after a bit then you just have to use your own discretion as to
whether to push it or keep poking people for comment.

Going through patches to figure out what should be applied is a pain in
the neck, particularly where there've been multiple updates. The
original submitter really ought to be in the best position to know what
to apply. Patches by the review team are often more complex than most
and therefore particular prone to this.

Cheers,

Ganesh


More information about the darcs-devel mailing list