On May 29, 2013, at 9:21 AM, Bem Jones-Bey <[email protected]> wrote:

> Would it be reasonable for webkit-patch to not clear flags on an attachement 
> when it obsoletes it if there's an r+? Maybe it doesn't actually matter (i.e. 
> I know I can still commit the patch), but it bothers me when it clears away 
> an r+ because I forgot to tell it --no-obsolete when I'm updating a patch for 
> nits.

Yes, we should make some change like this.

The problem is that it complicates writing a query that looks for 
reviewed-but-not-yet-landed patches. We need to solve that problem too.

It’s valuable to keep the history of what has been reviewed.

It’s valuable to conveniently find patches that have been reviewed but not 
landed.

-- Darin
_______________________________________________
webkit-dev mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-dev

Reply via email to