Closed
Bug 909767
Opened 12 years ago
Closed 11 years ago
clean up mobile_config.py
Categories
(Infrastructure & Operations Graveyard :: CIDuty, task)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: kmoir, Assigned: kmoir)
Details
Attachments
(2 files)
|
35.25 KB,
patch
|
Details | Diff | Splinter Review | |
|
26.67 KB,
patch
|
armenzg
:
review+
kmoir
:
checked-in+
|
Details | Diff | Splinter Review |
There are many pep8 errors in this file. Also, it could be cleaned up to make the assignment of test suites additive rather than subtractive, and thus easier to read.
|
Assignee | |
Updated•12 years ago
|
Assignee: nobody → kmoir
|
|
Assignee | |
Comment 2•12 years ago
|
||
Comment on attachment 799556 [details] [diff] [review]
pep8 compliant mobile_config.py patches
cancelling the review, I incorporated the patches for pep8 compliance in bug 829211
Attachment #799556 -
Flags: review?(pmoore)
|
|
Assignee | |
Comment 3•11 years ago
|
||
patches to fix some pep8 violations but not all
aki said on an another bug that we shouldn't fix all the line length ones because it makes the code less readable
Attachment #8401414 -
Flags: review?(armenzg)
|
||
Updated•11 years ago
|
Attachment #8401414 -
Flags: review?(armenzg) → review+
|
|
Assignee | |
Updated•11 years ago
|
Attachment #8401414 -
Flags: checked-in+
|
|
||
Comment 4•11 years ago
|
||
Live in production.
|
|
Assignee | |
Comment 5•11 years ago
|
||
fixed
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
|
||
Updated•7 years ago
|
Component: Platform Support → Buildduty
Product: Release Engineering → Infrastructure & Operations
|
||
Updated•6 years ago
|
Product: Infrastructure & Operations → Infrastructure & Operations Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•