[Buildbot-devel] Using pep8 on Buildbot source code

[Edmund Wong]

Vladimir Rutsky wrote:
> Hello!
>
> Recently Dustin merged changes that adds pep8
> (http://pep8.readthedocs.org/en/latest/intro.html) run on Buildbot
> source code on Travis and I hope it soon will be run on
> http://buildbot.buildbot.net (bugs http://trac.buildbot.net/ticket/2572,
> http://trac.buildbot.net/ticket/2573).

> 316     W291 trailing whitespace
>

Hi,

I've come across a quirk with pep8 while fixing W291 via pull request
#934 (which is closed.  vrutsky, Dustin said to use your autopep8
for the merge.).

https://github.com/ewongbb/buildbot/commit/5bc5c49d60ce76efa11c5cdb3af09facc4bb6f38

There's a space after 'Properties:' and pep8 tags it as a
W291 issue as given:

https://travis-ci.org/buildbot/buildbot/builds/12913927

I'm guessing this is a bug in pep8, though I'm not too sure.

Please clarify.

Edmund