Closed
Bug 777435
Opened 13 years ago
Closed 13 years ago
[all] ignore new pep8 errors until they're fixed
Categories
(Webtools Graveyard :: Elmo, defect, P2)
Tracking
(Not tracked)
RESOLVED
FIXED
2.3
People
(Reporter: Pike, Assigned: Pike)
Details
Attachments
(1 file)
|
365 bytes,
patch
|
peterbe
:
review+
|
Details | Diff | Splinter Review |
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•13 years ago
|
Assignee: nobody → l10n
|
|
Assignee | |
Comment 1•13 years ago
|
||
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)
|
|
||
Updated•13 years ago
|
Attachment #645824 -
Flags: review?(peterbe) → review+
|
|
Assignee | |
Updated•13 years ago
|
Priority: -- → P2
|
||
Comment 2•13 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•13 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
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2.3
Version: Trunk → 2.3
|
||
Updated•5 years ago
|
Product: Webtools → Webtools Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•