#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
------------------------------------------------------------------------------
October Webinars: Code for Performance
Free Intel webinars can help you accelerate application performance.
Explore tips for MPI, OpenMP, advanced profiling, and more. Get the most from
the latest Intel processors and coprocessors. See abstracts and register >
http://pubads.g.doubleclick.net/gampad/clk?id=60135031&iu=/4140/ostg.clktrk
_______________________________________________
Buildbot-commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/buildbot-commits
