[all] ignore new pep8 errors until they're fixed

RESOLVED FIXED in 2.3

Status

Webtools
Elmo
P2
normal
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: Pike, Assigned: Pike)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
Just updated my env, and it turns out that pep8 added a bunch of new errors about line continuations, E121-128.

I think we should add a .pep8 that ignores those, until we're actually addressing them.
(Assignee)

Updated

6 years ago
Assignee: nobody → l10n
(Assignee)

Comment 1

6 years ago
Created attachment 645824 [details] [diff] [review]
Add a .pep8 for now

These tests got enabled lately, and thus all our results regress badly, we should silence them until we tackle them.
Attachment #645824 - Flags: review?(peterbe)
Attachment #645824 - Flags: review?(peterbe) → review+
(Assignee)

Updated

6 years ago
Priority: -- → P2

Comment 2

6 years ago
Commit pushed to develop at https://github.com/mozilla/elmo

https://github.com/mozilla/elmo/commit/e764bdcd874388db6e78ae0a1266d20d500b5908
bug 777435, ignore continuation line pep 8 errors until we fix them, r=peterbe
(Assignee)

Comment 3

6 years ago
Landed, FIXED, I guess we should open new bugs on pep8 levels here that we want to switch back on on and individual level.
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2.3
Version: Trunk → 2.3
You need to log in before you can comment on or make changes to this bug.