Closed Bug 1337726 Opened 8 years ago Closed 8 years ago

Disable E129 pyflakes lint

Categories

(Tree Management :: Treeherder, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: jgraham, Assigned: jgraham)

References

Details

Attachments

(1 file)

This is the one that forces you to write something weird like if (something and something_else): pass just because it doesn't like the fact that "if (" is four characters. I don't think that the problem it's trying to solve is actually confusing, the alternative doesn't work well with my editor, and I think the code you wind up with trying to pacify the linter is really ugly to read. So let's just disable this check.
Attachment #8835551 - Flags: review?(emorley)
Attachment #8835551 - Flags: review?(emorley) → review+
Assignee: nobody → james
Commit pushed to master at https://github.com/mozilla/treeherder https://github.com/mozilla/treeherder/commit/067aad46bd4c2f1c298326bddeaef8cc118dfe34 Bug 1337726 - Disable E129 pyflakes lint. This demands a weird style when using if (something and something_else): which is not really helpful to code readability.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: