Closed Bug 664759 Opened 13 years ago Closed 7 years ago

socorro should use PEP8 for Python code

Categories

(Socorro :: General, task)

Product:
Socorro
Component:
General
Platform:
x86_64
Linux
Type:
task
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED DUPLICATE of bug 1377254

People

(Reporter: rhelmer, Assigned: willkg)

References

Details

Quoting lars from bug 654567 comment 18:

"It is ironic that I be the one to say this, but we ought to adopt PEP8 coding standards for all new code in the project.  I know someone is going to quote me and I'll never hear the end of it..."
It would be good to discuss updating our style guide at the offsite next week. 

PEP8 is nice because of we could hook the compliance tools into our tests to enforce style.
Consensus at the offsite seemed to be that this was a fine way to go. I've updated the code conventions guide to indicate that PEP 8 with 4 space indents is how we will proceed.
Blocks: 669464
Depends on: 678408
Depends on: 679123
Depends on: 686547
Component: Socorro → General
Product: Webtools → Socorro
Assignee: nobody → willkg
Mmm... this is great, but I think bug #1377254 has a better mission and is a superset of this bug: fix things so we can lint in CI.

Given that, I think I'm going to close dupe this bug to that bug.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.