<div dir="ltr"><div class="gmail_default" style=""><font face="verdana, sans-serif">In itself, the code clean up you have done is a good thing in the sense that you re-organized things and in my understanding, they look good now. In some of the teams I've been granted to work, there was a rule stating that whenever a dev would work on an enhancement/bugfix, he would create a separate ticket that would track all code refactoring related to his work. The strategy is quite different here and the suggestion was to have your refactoring be part of an enhancement/bugfix. The most valuable argument in favor of all the current reviewers is </font><span style="font-family:sans-serif;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline">Guido van Rossum's </span><span style="background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline"><font face="verdana, sans-serif">comment on the effect of having git blame made harder.<br>For some of the projects I'm currently working on, we have code cleanup teams where members are affected expressly to refactoring code. We might end up needing such things in the future but for now, it is wiser to leave the code as is and focus on fixing actual problems.</font></span></div><div><br></div>-- <br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><font size="2" style="font-family:arial black,sans-serif"><b>Q::Drone's Co-Founder and Co-CEO</b></font><br><span style="font-family:arial,helvetica,sans-serif"> </span><font size="2" style="font-family:arial,helvetica,sans-serif"><b>Hervé "Kyle" MUTOMBO</b></font></div><div><strong><font size="2"></font></strong> </div><div><strong><font size="2">Envoyé depuis le web.</font></strong></div></div></div></div>