<div dir="ltr">One thing @bizhang just found out is that you’ll probably need to rebase any old branches or open PRs to get the new .pep8speaks.yml config file. Otherwise, you may see some errors (e.g. line length > 79) that don’t apply.</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 Thu, Jul 13, 2017 at 1:56 PM, David Davis <span dir="ltr"><<a href="mailto:daviddavis@redhat.com" target="_blank">daviddavis@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">I wanted to send out an announcement about pep8speaks. A while ago we talked about using it and no one seemed opposed to the idea. So we thought we’d try it out for the main pulp repository. I’ve merged a config file for pep8speaks to master, 3.0-dev, and 2.13-dev. If you open a new PR against pulp you should see a comment from @pep8speaks that will tell you if your code conforms to pep8. Here is an example:<div><div><br></div><div><a href="https://github.com/pulp/pulp/pull/3084" target="_blank">https://github.com/pulp/pulp/<wbr>pull/3084</a></div><div><br></div><div><a href="https://github.com/pulp/pulp/pull/3084" target="_blank"></a>If we like this enough, we can enable it for other repositories and perhaps disable flake8 checking in our CI. However, I’d like for us to try it out for a couple weeks before we make those changes. S in the meantime, please respond with any thoughts or feedback you have on this.</div><div><br></div><div>Thank you.<span class="HOEnZb"><font color="#888888"><br clear="all"><div><div class="m_5561674057116705109gmail_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>
</blockquote></div><br></div>