On Mon, Sep 7, 2015 at 2:41 AM Andreas Sandberg <[email protected]>
wrote:

> I¹m not sure it¹s worth the overhead to add RB Ids in commits, but I¹m not
> opposed to the idea. I would like to see a system where we acknowledge
> reviewers though. The kernel uses a system where commits contain tags to
> keep track of who was involved with a patch
> (³Acked-by²/³Reviewed-by²/³Signed-off-by²/³Cc"). I think it would make
> sense for us to use a subset of those tags to acknowledge people who help
> out by reviewing code.
>

I like that idea too, but in the absence of automation, it's easier for
people to copy/paste a reviewboard URL via which others can look up all the
available information about the review than to have people manually
construct tags about who provided reviews etc.

Steve
_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to