<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Mar 2, 2015 at 1:28 PM, Stefano Maffulli <span dir="ltr"><<a href="mailto:stefano@openstack.org" target="_blank">stefano@openstack.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Mon, 2015-03-02 at 12:00 -0700, Doug Wiegley wrote:<br>
> Why do you feel the need to keep them?  Do your regularly look at<br>
> older patches? Do you know anyone that does?<br>
><br>
I don't think that's the point. The point is to try improving<br>
contributor's life by providing them one last useful comment before<br>
ignoring their contribution for good.<br>
<br></blockquote><div>To me, the abandon does this. It's basically giving a good reason for why it's being abandoned (4 weeks old, one -2. Or, 4 weeks old, no new comments), with a nice message on how to re-enable the patch and get fresh test results.<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Tom gave a good suggestion IMHO and I've found a couple of cases where<br>
maybe if someone reached out to the contributor and offered some help<br>
probably their patch would have merged (or they would have learned a<br>
useful, explicit lesson). Instead auto-abandon has *implicit*<br>
connotation: a contributor may never know exactly why that patch was<br>
abandoned.<br>
<br>
I suggest to give a second look at Tom's proposal because I think it<br>
doesn't add any more workload on reviewers but provides for a clean exit<br>
to a lagging changeset.<br>
<br>
/stef<br>
<br>
<br>
<br>
__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</blockquote></div><br></div></div>