<div dir="ltr"><div><a href="https://github.com/sugarlabs/sugar/pull/66">https://github.com/sugarlabs/sugar/pull/66</a><br><br></div>A quick review would be appreciated, to unbreak the build.<br></div><div class="gmail_extra">
<br><br><div class="gmail_quote">On 17 May 2013 21:41, Daniel Narvaez <span dir="ltr"><<a href="mailto:dwnarvaez@gmail.com" target="_blank">dwnarvaez@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span style="font-family:'.HelveticaNeueUI';white-space:nowrap"><a href="http://buildbot.sugarlabs.org/builders/raring-amd64-quick/builds/9/steps/shell_3/logs/stdio" target="_blank">http://buildbot.sugarlabs.org/builders/raring-amd64-quick/builds/9/steps/shell_3/logs/stdio</a></span><div>

<span style="font-family:'.HelveticaNeueUI';white-space:nowrap"><br></span></div><div><font face=".HelveticaNeueUI"><span style="white-space:nowrap">config.py is autogenerated, so lines <span></span>can be arbitrary long.</span></font></div>

<div><font face=".HelveticaNeueUI"><span style="white-space:nowrap"><br></span></font></div><div><font face=".HelveticaNeueUI"><span style="white-space:nowrap">Perhaps the simplest fix it just to use --exclude.</span></font></div>

<div><font face=".HelveticaNeueUI"><span style="white-space:nowrap">We could also consider switching to flake8 to run both pyflakes and pep8 and be able to skip certain lines.</span></font></div><div><font face=".HelveticaNeueUI"><span style="white-space:nowrap">Maybe it's overkill though, since we are not needing that feature other than for this case.</span></font></div>

<div><font face=".HelveticaNeueUI"><span style="white-space:nowrap"> (I sort of like the asserts. To silence pyflake when necessary.)</span></font></div><span class="HOEnZb"><font color="#888888"><br><br>-- <br>Daniel Narvaez<br>
<br>
</font></span></blockquote></div><br><br clear="all"><br>-- <br>Daniel Narvaez<br>
</div>