[DISPATCH-1974] Fix and remove ignores in flake8 configuration

• Type: Improvement
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Affects Version/s: 1.16.0
• Fix Version/s: Backlog
• Assignee: Unassigned
• Reporter: Jiri Daněk
• Created: 21/Feb/21 18:24
• Updated: 30/Jan/22 22:47

Description

Fixes with autopep8 in DISPATCH-1814 resolved many flake8 warnings. There are still warnings left that require human consideration and fixing.

For example, usage of bare except in the code.

This must be fixed manually. Fixed issues should then be removed from the tox.ini flake8 ignore section.

List of "useful" suppressed warnings

    # H101: Use TODO(NAME)
    # H202: assertRaises Exception too broad
    # E722: do not use bare 'except'
    # H201: no 'except:' at least use 'except Exception:'
    # F811: redefinition of unused 'Timeout' from line 29
    # F405: may be undefined, or defined from star imports: datetime

Issue Links

• depends upon: DISPATCH-1814 Apply autofixes to resolve some flake8 code formatting issues