Closed Bug 1337726 Opened 7 years ago Closed 7 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: 7 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: