<div dir="ltr">On 8 May 2013 06:02, William Orr <span dir="ltr"><<a href="mailto:will@worrbase.com" target="_blank">will@worrbase.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I've done some initial work on both sugar and sugar-toolkit-gtk3 with respect to PEP8. I've also added support for PEP8 in make check for both packages.<br>
<br>
Pull requests are here:<br>
<a href="https://github.com/sugarlabs/sugar-toolkit-gtk3/pull/25" target="_blank">https://github.com/sugarlabs/<u></u>sugar-toolkit-gtk3/pull/25</a><br>
<a href="https://github.com/sugarlabs/sugar/pull/49" target="_blank">https://github.com/sugarlabs/<u></u>sugar/pull/49</a><br>
<br>
Let me know if I need to make any changes.<br></blockquote><div><br></div><div>Awesome! Thanks for working on this!<br><br></div><div>I didn't go through all the changes yet, but I have one general concern. You have disabled E501 (line too long), which would be fine as an intermediate step, though you seem to be regressing line length when fixing the other issues. Perhaps it make sense to do the E501 changes together so that we avoid regressions?<br>
<br>If you end up reworking the patches it might also make sense to split them up a bit more (even arbitrarily) just to review/land more gradually and diminish a bit the risk of conflicts etc. I don't expect the review to be particularly complicated, though there are a few odd cases with line continuations that we will need to check/discuss/tweak.<br>
</div></div></div></div>