<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small">Thanks so much for working this Zane, it's much appreciated! :-)</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">On Tue, Dec 3, 2019 at 2:51 PM Zane Bitter <<a href="mailto:zbitter@redhat.com">zbitter@redhat.com</a>> wrote:</div></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
I added a column for "Core Reviewer comments" as well.<br></blockquote><div><br></div><div class="gmail_default" style="font-size:small">I hijacked this column for my amendments too.</div><div class="gmail_default" style="font-size:small"></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">> *_Rebase + Merge_*<br><br>
Would you have time to start going through these and rebase or recheck <br>
them (as required), then provide a list somewhere of priority reviews <br>
(i.e. those that rebased easily and passed tests) for cores to go <br>
through? It should be pretty easy to merge a good number of these quite <br>
quickly.<br></blockquote><div> </div><div class="gmail_default" style="font-size:small">I went through a big blitz of all those rebase + merge patches today; as expected many of them were trivial rebases but a lot were very broken due to their age.  I fixed up each and every single one, and even iterated on core review suggestions on a some.  They're all in good shape for review / pushing through the merge now.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">When cores have time it would be great to get those reviewed and hopefully workflowed.</div><div class="gmail_default" style="font-size:small"></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> *_Research_*<br>
> <br>
> 76 patches are sufficiently complex that they'll need a much closer <br>
> look.  Some of these patches are in a seemingly "finished" state, some <br>
> are a way off.  Some have unanswered concerns from core review and have <br>
> been left dangling. *If you're the original developer or otherwise <br>
> interested in working these patches through to completion, please do get <br>
> involved.*<br>
<br>
As a first step it'd be great if we could extract from this a list of <br>
stuff that looks promising but hasn't been reviewed by a core. Those <br>
would be the next highest priority to review I'd expect.<br></blockquote><div><br></div><div class="gmail_default" style="font-size:small">Absolutely - that's a great idea!  I'll handle this.</div><div class="gmail_default" style="font-size:small"></div><div class="gmail_default" style="font-size:small">This is a very rewarding journey, and as I work the last few days before I go on my winter solstice break, it's fantastic for reflection too.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Looking forward to continuing this endeavour into the new year.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">For those who celebrate, enjoy your holidays.  For everyone else, keep being excellent! :-)</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Cheers,</div><div class="gmail_default" style="font-size:small">David</div></div></div>