application is not pep8 compliant

Bug #1212015 reported by Leo Arias
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ubuntu Weather App
Fix Released
Low
Corey Goldberg

Bug Description

$ pep8 ../tests/autopilot/ubuntu_weather_app/emulators.py:31:1: E302 expected 2 blank lines, found 1
./tests/autopilot/ubuntu_weather_app/emulators.py:34:31: W291 trailing whitespace
./tests/autopilot/ubuntu_weather_app/emulators.py:35:49: W291 trailing whitespace
./tests/autopilot/ubuntu_weather_app/emulators.py:38:34: W291 trailing whitespace
./tests/autopilot/ubuntu_weather_app/emulators.py:39:41: W291 trailing whitespace
./tests/autopilot/ubuntu_weather_app/tests/__init__.py:24:1: E302 expected 2 blank lines, found 1
./tests/autopilot/ubuntu_weather_app/tests/__init__.py:64:80: E501 line too long (85 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/__init__.py:87:1: E302 expected 2 blank lines, found 1
./tests/autopilot/ubuntu_weather_app/tests/__init__.py:91:80: E501 line too long (84 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/__init__.py:95:80: E501 line too long (84 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/__init__.py:97:1: E302 expected 2 blank lines, found 1
./tests/autopilot/ubuntu_weather_app/tests/__init__.py:104:80: E501 line too long (108 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/__init__.py:108:18: E201 whitespace after '['
./tests/autopilot/ubuntu_weather_app/tests/__init__.py:108:80: E501 line too long (87 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/__init__.py:108:86: E202 whitespace before ']'
./tests/autopilot/ubuntu_weather_app/tests/__init__.py:140:80: E501 line too long (122 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/__init__.py:143:1: W391 blank line at end of file
./tests/autopilot/ubuntu_weather_app/tests/test_locationmanager.py:31:80: E501 line too long (93 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/test_locationmanager.py:34:33: E712 comparison to True should be 'if cond is not True:' or 'if not cond:'
./tests/autopilot/ubuntu_weather_app/tests/test_locationmanager.py:46:80: E501 line too long (84 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/test_locationmanager.py:53:80: E501 line too long (84 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/test_locationmanager.py:77:80: E501 line too long (81 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/test_locationmanager.py:78:80: E501 line too long (93 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/test_locationmanager.py:79:80: E501 line too long (87 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/test_locationmanager.py:80:80: E501 line too long (87 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/test_locationmanager.py:88:80: E501 line too long (84 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/test_locationmanager.py:90:80: E501 line too long (80 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/test_locationmanager.py:96:80: E501 line too long (91 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/test_locationmanager.py:102:64: E231 missing whitespace after ','
./tests/autopilot/ubuntu_weather_app/tests/test_locationmanager.py:114:80: E501 line too long (81 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/test_locationmanager.py:115:80: E501 line too long (93 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/test_locationmanager.py:116:80: E501 line too long (87 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/test_locationmanager.py:117:80: E501 line too long (87 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/test_locationmanager.py:119:80: E501 line too long (86 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/test_locationmanager.py:132:80: E501 line too long (93 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/test_locationmanager.py:133:80: E501 line too long (87 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/test_locationmanager.py:135:80: E501 line too long (86 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/test_locationmanager.py:143:80: E501 line too long (84 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/test_locationmanager.py:155:80: E501 line too long (81 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/test_locationmanager.py:156:80: E501 line too long (93 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/test_locationmanager.py:157:80: E501 line too long (87 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/test_locationmanager.py:162:80: E501 line too long (82 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/test_locationmanager.py:179:80: E501 line too long (90 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/test_locationmanager.py:192:80: E501 line too long (86 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/test_locationmanager.py:202:64: E231 missing whitespace after ','
./tests/autopilot/ubuntu_weather_app/tests/test_locationmanager.py:211:80: E501 line too long (91 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/test_locationmanager.py:213:64: E231 missing whitespace after ','
./tests/autopilot/ubuntu_weather_app/tests/test_locationmanager.py:214:80: E501 line too long (84 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/test_locationmanager.py:223:64: E231 missing whitespace after ','
./tests/autopilot/ubuntu_weather_app/tests/test_locationmanager.py:234:64: E231 missing whitespace after ','
./tests/autopilot/ubuntu_weather_app/tests/test_locationmanager.py:236:1: W391 blank line at end of file
./tests/autopilot/ubuntu_weather_app/tests/test_mainview.py:22:1: E302 expected 2 blank lines, found 1
./tests/autopilot/ubuntu_weather_app/tests/test_mainview.py:38:80: E501 line too long (175 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/test_mainview.py:44:15: E222 multiple spaces after operator
./tests/autopilot/ubuntu_weather_app/tests/test_mainview.py:45:64: E231 missing whitespace after ','
./tests/autopilot/ubuntu_weather_app/tests/test_mainview.py:59:58: E231 missing whitespace after ','
./tests/autopilot/ubuntu_weather_app/tests/test_mainview.py:66:80: E501 line too long (91 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/test_mainview.py:67:80: E501 line too long (87 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/test_mainview.py:70:63: E231 missing whitespace after ','
./tests/autopilot/ubuntu_weather_app/tests/test_settings.py:18:1: E302 expected 2 blank lines, found 1
./tests/autopilot/ubuntu_weather_app/tests/test_settings.py:32:80: E501 line too long (84 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/test_settings.py:39:80: E501 line too long (85 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/test_settings.py:54:80: E501 line too long (86 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/test_settings.py:67:80: E501 line too long (91 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/test_settings.py:73:80: E501 line too long (86 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/test_settings.py:90:80: E501 line too long (91 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/test_settings.py:100:80: E501 line too long (86 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/test_settings.py:115:1: W391 blank line at end of file
./tests/autopilot/ubuntu_weather_app/tests/weatherdata.py:9:80: E501 line too long (4182 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/weatherdata.py:11:80: E501 line too long (4368 > 79 characters)
./tests/autopilot/ubuntu_weather_app/tests/weatherdata.py:12:1: W391 blank line at end of file

Leo Arias (elopio)
Changed in ubuntu-weather-app:
status: New → In Progress
assignee: nobody → Leo Arias (elopio)
David Planella (dpm)
Changed in ubuntu-weather-app:
importance: Undecided → Low
Changed in ubuntu-weather-app:
assignee: Leo Arias (elopio) → Corey Goldberg (coreygoldberg)
Revision history for this message
Martin Borho (martin-borho) wrote :

I guess, this one can be marked as fixed, as pep8 checks are now happening in jenkins

Revision history for this message
Leo Arias (elopio) wrote :

Yes, thanks Martin.

Changed in ubuntu-weather-app:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.