On Mon, Nov 16, 2009 at 2:16 PM, Ganesh Sittampalam <[email protected]> wrote:

> Hi,
>
> Generally when reviewing patches, I read them directly, perhaps have a look
> at the source they apply to, and then decide that they're fine so long as
> the tests pass. Actually keeping track of them between that point and
> running the tests and pushing is a bit of a pain, and there's a cognitive
> shift that feels quite disruptive.
>
> I wonder if we can add a new status to the patch tracker to reflect the
> state of "had human review, needs standard tests run"? It would make it
> easier to do different jobs in batch mode, and it would also allow different
> people to do the review and the actual push.
>

+1

I support this on the basis of this last sentence alone!

Jason
_______________________________________________
darcs-users mailing list
[email protected]
http://lists.osuosl.org/mailman/listinfo/darcs-users

Reply via email to