Status

task
RESOLVED FIXED
6 years ago
a year ago

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
Last Resolved: 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.