<div dir="ltr">+1 from me. <div><br></div><div>At the very least, we could try it out for a while and see how it goes.</div></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><br></div><div>David<br></div></div></div></div></div></div></div></div>
<br><div class="gmail_quote">On Mon, Jul 3, 2017 at 1:05 PM, Brian Bouterse <span dir="ltr"><<a href="mailto:bbouters@redhat.com" target="_blank">bbouters@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">@daviddavis thanks for the flake8 fixes.<br><br>I was thinking we should enable pep8speaks [1] for all of our branches and repos. We can still have Travis or other places run flake8 but that would put it right in the PR comments.<br><br>[1]: <a href="https://github.com/OrkoHunter/pep8speaks" target="_blank">https://github.com/OrkoHunter/<wbr>pep8speaks</a><br></div><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="h5">On Mon, Jul 3, 2017 at 12:03 PM, David Davis <span dir="ltr"><<a href="mailto:daviddavis@redhat.com" target="_blank">daviddavis@redhat.com</a>></span> wrote:<br></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5"><div dir="ltr">I just merged a PR to fix flake8 on the 3.0-dev branch of Pulp[0]. It also fixes the flake8 warnings that came up while flake8 was not running in Travis. <div><br></div><div>If you’re developing Pulp 3, please rebase your branch with 3.0-dev from upstream. Also for bonus points, check the Travis logs for your PRs to make sure flake8 is running on the entire codebase and not via “git diff…”. <div><br></div><div>Let me know if you need any help.<div><br></div><div>Thanks!<br><div><br></div><div>[0] <a href="https://github.com/pulp/pulp/pull/3019" target="_blank">https://github.com/pulp/pu<wbr>lp/pull/3019</a><span class="m_-229016202700497833HOEnZb"><font color="#888888"><br clear="all"><div><div class="m_-229016202700497833m_1439907375768066931gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><br></div><div>David<br></div></div></div></div></div></div></div></div>
</font></span></div></div></div></div></div>
<br></div></div>______________________________<wbr>_________________<br>
Pulp-dev mailing list<br>
<a href="mailto:Pulp-dev@redhat.com" target="_blank">Pulp-dev@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/pulp-dev" rel="noreferrer" target="_blank">https://www.redhat.com/mailman<wbr>/listinfo/pulp-dev</a><br>
<br></blockquote></div><br></div>
</blockquote></div><br></div>