Status

task
RESOLVED FIXED
6 years ago
Last year

People

(Reporter: kmoir, Assigned: kmoir)

Tracking

Details

Attachments

(2 attachments)

Assignee

Description

6 years ago
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

6 years ago
Assignee: nobody → kmoir
Assignee

Comment 1

6 years ago
Tested in staging
Attachment #799556 - Flags: review?(pmoore)
Assignee

Comment 2

6 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

5 years ago
Posted patch patchSplinter Review
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

5 years ago
Attachment #8401414 - Flags: review?(armenzg) → review+
Assignee

Updated

5 years ago
Attachment #8401414 - Flags: checked-in+

Comment 4

5 years ago
Live in production.
Assignee

Comment 5

5 years ago
fixed
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Component: Platform Support → Buildduty
Product: Release Engineering → Infrastructure & Operations
You need to log in before you can comment on or make changes to this bug.