[Buildbot-commits] [Buildbot] #2572: Use pep8 on Buildbot source

[Buildbot trac]

#2572: Use pep8 on Buildbot source
----------------------+------------------------
Reporter:  rutsky     |       Owner:
    Type:  undecided  |      Status:  new
Priority:  major      |   Milestone:  undecided
 Version:  0.8.8      |  Resolution:
Keywords:             |
----------------------+------------------------

Comment (by rutsky):

 Attached output of `pep8 --first .`, it also duplicated
 [https://gist.github.com/vrutsky/7049271 here].

 Detailed log of all pep8 errors that occures in Buildbot sources:
 https://gist.github.com/vrutsky/7049307 .

 I made pull requests for [https://github.com/buildbot/buildbot/pull/921
 master] and [https://github.com/buildbot/buildbot/pull/923 nine] branches
 that:

 1) adds `pep8` target to `Makefile`;

 2) adds all currently reported pep8 errors to ignore list (so that we can
 incrementally fix these issues one by one, without breaking whole build);

 3) runs `make pep8` on Travis;

 4) fixes E101 (indentation contains mixed spaces and tabs), E111
 (indentation is not a multiple of four) pep8 errors.

-- 
Ticket URL: <http://trac.buildbot.net/ticket/2572#comment:1>
Buildbot <http://buildbot.net/>
Buildbot: build/test automation