Code Style and PEP8

Bug #1280660 reported by Pavel Savchenko
14
This bug affects 3 people
Affects Status Importance Assigned to Milestone
ladon
New
Undecided
Unassigned

Bug Description

Guido says -> use PEP8 ! http://www.python.org/dev/peps/pep-0008/

But seriously, mixed tabs and spaces is a recipe for disaster (talking from experience here), I suggest at least replacing all tabs and other indents with a constant 4 spaces per indent level (as PEP8 dictates). I'd also add spaces between arguments and a space between operator and operands.

This would not take too much effort and I'd love to submit a patch if this is acceptable.

Revision history for this message
André (andre-miras) wrote :

I second this. I worked on the code base recently and thought the same. I would like to contribute, but I think jsgaarde is too busy to review and merge changes. Perhaps it would be easier/faster for him to review and merge if he were on github.

Revision history for this message
Pavel Savchenko (asfaltboy) wrote :

Thanks André, I sent a bump to Jakob via launchpad's internal messaging system, lets wait for his reply.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.