Code review comment for lp:~macslow/unity8/shellRotation

Revision history for this message
MichaƂ Sawicz (saviq) wrote :

Please fix the flake8 issues:

$ python3 -m flake8 tests
tests/autopilot/unity8/fixture_setup.py:43:1: F401 'get_lib_path' imported but unused
tests/autopilot/unity8/fixture_setup.py:43:1: F811 redefinition of unused 'get_binary_path' from line 31
tests/autopilot/unity8/fixture_setup.py:53:1: E302 expected 2 blank lines, found 1
tests/autopilot/unity8/fixture_setup.py:89:80: E501 line too long (83 > 79 characters)
tests/autopilot/unity8/fixture_setup.py:140:80: E501 line too long (80 > 79 characters)
tests/autopilot/unity8/fixture_setup.py:141:80: E501 line too long (87 > 79 characters)
tests/autopilot/unity8/process_helpers.py:110:19: F821 undefined name '_get_unity_pid'
tests/autopilot/unity8/indicators/tests/test_display_indicator.py:19:1: F401 'platform' imported but unused
tests/autopilot/unity8/indicators/tests/test_display_indicator.py:21:1: F401 'process_helpers' imported but unused
tests/autopilot/unity8/indicators/tests/test_display_indicator.py:46:5: E303 too many blank lines (2)

review: Needs Fixing

« Back to merge proposal