[ovirt-devel] Abandoning old patches

Michal Skrivanek michal.skrivanek at redhat.com
Tue Sep 8 08:41:35 UTC 2015


On 7 Sep 2015, at 15:35, Tal Nisan wrote:

> Why did we choose 300 and 600? I think it's way too much, I'd go for emailing after 30 days and for abandoning after 60 days.

it's coming…;-)
this is just a first huge cleanup of really ridiculously outdated stuff


> It's reversible anyway, abandoned patches can always be restored by the author.
> 
> On 09/05/2015 01:18 PM, Allon Mureinik wrote:
>> Hi guys,
>> 
>> Gerrit is our front yard, and we'd like to keep it clean, tidy, and welcoming to newcommers.
>> 
>> We have a jenkins job waiting to act as a roomba just waiting to be switched on (thanks Eyal!).
>> It will send out an email about patches being stale for over 300 days and will force abandon patches stale over 600 old.
>> 
>> What we need you to do:
>> Go over your patches and make sure you haven't missed anything. If you have a path that old that for some obscure reason still has merit, just comment on it that it's still relevant to bump its last modification date (or better yet - answer the review, rebase it, and push for it to get merged).
>> 
>> To search for such patches, you can use the "status:open age:300d" search string in gerrit.
>> So, e.g, for your own patches, you could use "status:open age:300d owner:self", or as a quick link:
>> https://gerrit.ovirt.org/#/q/status:open+age:300d+owner:self
>> 
>> 
>> Thanks,
>> Your friendly neighbourhood cleanup patrol
>> 
>> 
>> _______________________________________________
>> Devel mailing list
>> Devel at ovirt.org
>> http://lists.ovirt.org/mailman/listinfo/devel
> 
> _______________________________________________
> Devel mailing list
> Devel at ovirt.org
> http://lists.ovirt.org/mailman/listinfo/devel

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.ovirt.org/pipermail/devel/attachments/20150908/b2d8ab57/attachment-0001.html>


More information about the Devel mailing list