[Pulp-dev] pep8speaks

David Davis daviddavis at redhat.com
Thu Jul 13 18:22:18 UTC 2017


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.


David

On Thu, Jul 13, 2017 at 1:56 PM, David Davis <daviddavis at redhat.com> wrote:

> 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:
>
> https://github.com/pulp/pulp/pull/3084
>
> <https://github.com/pulp/pulp/pull/3084>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.
>
> Thank you.
>
> David
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/pulp-dev/attachments/20170713/0acee9b9/attachment.htm>


More information about the Pulp-dev mailing list