Abandoning old patches
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
This is a multi-part message in MIME format. --------------090500090808080100010801 Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit 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 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@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
--------------090500090808080100010801 Content-Type: text/html; charset=windows-1252 Content-Transfer-Encoding: 7bit <html> <head> <meta content="text/html; charset=windows-1252" http-equiv="Content-Type"> </head> <body text="#000000" bgcolor="#FFFFFF"> 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.<br> It's reversible anyway, abandoned patches can always be restored by the author.<br> <br> <div class="moz-cite-prefix">On 09/05/2015 01:18 PM, Allon Mureinik wrote:<br> </div> <blockquote cite="mid:CADgXf3xysrY9Y-J0pufNOjVOs3Qu=vwuAXMt2ENZgdGqDPW8_g@mail.gmail.com" type="cite"> <div dir="ltr"> <div class="gmail_default" style="font-family:monospace,monospace">Hi guys,</div> <div class="gmail_default" style="font-family:monospace,monospace"><br> </div> <div class="gmail_default" style="font-family:monospace,monospace">Gerrit is our front yard, and we'd like to keep it clean, tidy, and welcoming to newcommers.</div> <div class="gmail_default" style="font-family:monospace,monospace"><br> </div> <div class="gmail_default" style="font-family:monospace,monospace">We have a jenkins job waiting to act as a roomba just waiting to be switched on (thanks Eyal!).</div> <div class="gmail_default" style="font-family:monospace,monospace">It will send out an email about patches being stale for over 300 days and will force abandon patches stale over 600 old.</div> <div class="gmail_default" style="font-family:monospace,monospace"><br> </div> <div class="gmail_default" style="font-family:monospace,monospace">What we need you to do:</div> <div class="gmail_default" style="font-family:monospace,monospace">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).</div> <div class="gmail_default" style="font-family:monospace,monospace"><br> </div> <div class="gmail_default" style="font-family:monospace,monospace">To search for such patches, you can use the "status:open age:300d" search string in gerrit.</div> <div class="gmail_default" style="font-family:monospace,monospace">So, e.g, for your own patches, you could use "status:open age:300d owner:self", or as a quick link:</div> <div class="gmail_default" style=""><font face="monospace, monospace"><a moz-do-not-send="true" href="https://gerrit.ovirt.org/#/q/status:open+age:300d+owner:self">https://gerrit.ovirt.org/#/q/status:open+age:300d+owner:self</a></font><br> </div> <div class="gmail_default" style=""><font face="monospace, monospace"><br> </font></div> <div class="gmail_default" style=""><font face="monospace, monospace"><br> </font></div> <div class="gmail_default" style=""><font face="monospace, monospace">Thanks,</font></div> <div class="gmail_default" style=""><font face="monospace">Your friendly neighbourhood cleanup patrol</font></div> </div> <br> <fieldset class="mimeAttachmentHeader"></fieldset> <br> <pre wrap="">_______________________________________________ Devel mailing list <a class="moz-txt-link-abbreviated" href="mailto:Devel@ovirt.org">Devel@ovirt.org</a> <a class="moz-txt-link-freetext" href="http://lists.ovirt.org/mailman/listinfo/devel">http://lists.ovirt.org/mailman/listinfo/devel</a></pre> </blockquote> <br> </body> </html> --------------090500090808080100010801--
--Apple-Mail=_0BDD35EA-89FA-43D9-BB22-6D9BA4B78AAC Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=windows-1252 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 reversible anyway, abandoned patches can always be restored by =
=20 On 09/05/2015 01:18 PM, Allon Mureinik wrote:
Hi guys, =20 Gerrit is our front yard, and we'd like to keep it clean, tidy, and = welcoming to newcommers. =20 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. =20 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 =
it's coming=85;-) this is just a first huge cleanup of really ridiculously outdated stuff the author. push for it to get merged).
=20 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 =20 =20 Thanks, Your friendly neighbourhood cleanup patrol =20 =20 _______________________________________________ Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel =20
Devel mailing list Devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/devel
--Apple-Mail=_0BDD35EA-89FA-43D9-BB22-6D9BA4B78AAC Content-Transfer-Encoding: quoted-printable Content-Type: text/html; charset=windows-1252 <html><head></head><body style=3D"word-wrap: break-word; = -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; = "><br><div><div>On 7 Sep 2015, at 15:35, Tal Nisan wrote:</div><br = class=3D"Apple-interchange-newline"><blockquote type=3D"cite"> =20 <meta content=3D"text/html; charset=3Dwindows-1252" = http-equiv=3D"Content-Type"> =20 <div text=3D"#000000" bgcolor=3D"#FFFFFF"> 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.<br></div></blockquote><div><br></div>it's = coming=85;-)</div><div>this is just a first huge cleanup of really = ridiculously outdated stuff</div><div><br></div><div><br><blockquote = type=3D"cite"><div text=3D"#000000" bgcolor=3D"#FFFFFF"> It's reversible anyway, abandoned patches can always be restored by the author.<br> <br> <div class=3D"moz-cite-prefix">On 09/05/2015 01:18 PM, Allon = Mureinik wrote:<br> </div> <blockquote = cite=3D"mid:CADgXf3xysrY9Y-J0pufNOjVOs3Qu=3DvwuAXMt2ENZgdGqDPW8_g@mail.gma= il.com" type=3D"cite"> <div dir=3D"ltr"> <div class=3D"gmail_default" = style=3D"font-family:monospace,monospace">Hi guys,</div> <div class=3D"gmail_default" = style=3D"font-family:monospace,monospace"><br> </div> <div class=3D"gmail_default" = style=3D"font-family:monospace,monospace">Gerrit is our front yard, and we'd like to keep it clean, tidy, and welcoming to newcommers.</div> <div class=3D"gmail_default" = style=3D"font-family:monospace,monospace"><br> </div> <div class=3D"gmail_default" = style=3D"font-family:monospace,monospace">We have a jenkins job waiting to act as a roomba just waiting to be switched on (thanks Eyal!).</div> <div class=3D"gmail_default" = style=3D"font-family:monospace,monospace">It will send out an email about patches being stale for over 300 days and will force abandon patches stale over 600 old.</div> <div class=3D"gmail_default" = style=3D"font-family:monospace,monospace"><br> </div> <div class=3D"gmail_default" = style=3D"font-family:monospace,monospace">What we need you to do:</div> <div class=3D"gmail_default" = style=3D"font-family:monospace,monospace">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).</div> <div class=3D"gmail_default" = style=3D"font-family:monospace,monospace"><br> </div> <div class=3D"gmail_default" = style=3D"font-family:monospace,monospace">To search for such patches, you can use the "status:open age:300d" search string in gerrit.</div> <div class=3D"gmail_default" = style=3D"font-family:monospace,monospace">So, e.g, for your own patches, you could use "status:open age:300d owner:self", or as a quick link:</div> <div class=3D"gmail_default" style=3D""><font face=3D"monospace, monospace"><a moz-do-not-send=3D"true" = href=3D"https://gerrit.ovirt.org/#/q/status:open+age:300d+owner:self">http= s://gerrit.ovirt.org/#/q/status:open+age:300d+owner:self</a></font><br> </div> <div class=3D"gmail_default" style=3D""><font face=3D"monospace, monospace"><br> </font></div> <div class=3D"gmail_default" style=3D""><font face=3D"monospace, monospace"><br> </font></div> <div class=3D"gmail_default" style=3D""><font face=3D"monospace, monospace">Thanks,</font></div> <div class=3D"gmail_default" style=3D""><font = face=3D"monospace">Your friendly neighbourhood cleanup patrol</font></div> </div> <br> <fieldset class=3D"mimeAttachmentHeader"></fieldset> <br> <pre wrap=3D"">_______________________________________________ Devel mailing list <a class=3D"moz-txt-link-abbreviated" = href=3D"mailto:Devel@ovirt.org">Devel@ovirt.org</a> <a class=3D"moz-txt-link-freetext" = href=3D"http://lists.ovirt.org/mailman/listinfo/devel">http://lists.ovirt.= org/mailman/listinfo/devel</a></pre> </blockquote> <br> </div> _______________________________________________<br>Devel mailing = list<br><a = href=3D"mailto:Devel@ovirt.org">Devel@ovirt.org</a><br>http://lists.ovirt.= org/mailman/listinfo/devel</blockquote></div><br></body></html>= --Apple-Mail=_0BDD35EA-89FA-43D9-BB22-6D9BA4B78AAC--
participants (3)
-
Allon Mureinik -
Michal Skrivanek -
Tal Nisan