Skip to content

Fix some flake8 warnings

Benjamin Tissoires requested to merge bentiss/hid-tools:wip/flake8 into master

Should be pretty straightforward.

Running a bare flake8 with --ignore E501,W504 now gives:

./tests/test_multitouch.py:37:25: E241 multiple spaces after ':'
./tests/test_multitouch.py:38:25: E241 multiple spaces after ':'
./tests/test_multitouch.py:39:15: E241 multiple spaces after ':'
./tests/test_multitouch.py:40:29: E241 multiple spaces after ':'
./tests/test_multitouch.py:41:20: E241 multiple spaces after ':'
./tests/test_multitouch.py:42:24: E241 multiple spaces after ':'
./tests/test_multitouch.py:43:27: E241 multiple spaces after ':'
./tests/test_multitouch.py:44:18: E241 multiple spaces after ':'
./tests/test_multitouch.py:45:35: E241 multiple spaces after ':'
./tests/test_multitouch.py:46:15: E241 multiple spaces after ':'
./tests/test_multitouch.py:47:25: E241 multiple spaces after ':'
./tests/test_multitouch.py:48:16: E241 multiple spaces after ':'
./tests/test_multitouch.py:49:28: E241 multiple spaces after ':'
./tests/test_multitouch.py:50:25: E241 multiple spaces after ':'
./tests/test_multitouch.py:51:28: E241 multiple spaces after ':'
./tests/test_multitouch.py:52:26: E241 multiple spaces after ':'
./tests/test_multitouch.py:53:22: E241 multiple spaces after ':'
./tests/test_multitouch.py:54:22: E241 multiple spaces after ':'
./tests/test_multitouch.py:55:24: E241 multiple spaces after ':'
./hidtools/hid.py:741:13: E722 do not use bare 'except'
./hidtools/hut.py:402:21: E722 do not use bare 'except'

So that is only 2 bare 'except' that would be nice to fix and probably we might just add # noqa: E241 for the bloc in test_multitouch.py.

The 2 last bare 'except' are not that trivial to fix, so it might require more testing that just reading the code.

Merge request reports