On 15/07/09 at 23:56 +0200, Raphael Hertzog wrote:
> On Fri, 03 Jul 2009, Jon Dowland wrote:
> > One thing I would like to see patch metadata help
> > facilitate is patch review. At the moment the "Reviewed-by"
> > header proposed would allow a tool to ensure at least two
> > sets of eyeballs had seen a patch; however, patches can go
> > stale. I think some chronological information is needed
> > alongside the review. I propose a "Last-Reviewed" header to
> > capture this information.
> 
> Only Sean Penny acked this suggestion. What do other people think of it?

I think that this information should be stored outside of the patch (in
the history of a VCS, for example).
-- 
| Lucas Nussbaum
| lu...@lucas-nussbaum.net   http://www.lucas-nussbaum.net/ |
| jabber: lu...@nussbaum.fr             GPG: 1024D/023B3F4F |


-- 
To UNSUBSCRIBE, email to debian-devel-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to