Autopilot doesn't follow pep8
Bug #1201057 reported by
Leo Arias
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Autopilot |
Fix Released
|
Undecided
|
Leo Arias | ||
autopilot (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
One of the best things of python code is that it is pretty. If we follow pep8 [1], it's even prettier. Also, it easier to read for people used to follow those conventions.
Related branches
lp:~elopio/autopilot/fix1201057-pep8-1
- Thomi Richards (community): Approve
- PS Jenkins bot: Approve (continuous-integration)
-
Diff: 598 lines (+145/-117)3 files modifiedautopilot/input/__init__.py (+86/-78)
autopilot/input/_common.py (+21/-10)
autopilot/input/_uinput.py (+38/-29)
lp:~elopio/autopilot/fix1201057-pep8-2
- Thomi Richards (community): Approve
- PS Jenkins bot: Approve (continuous-integration)
-
Diff: 619 lines (+160/-120)3 files modifiedautopilot/matchers/__init__.py (+25/-18)
autopilot/process/__init__.py (+49/-47)
autopilot/process/_bamf.py (+86/-55)
lp:~elopio/autopilot/fix1201057-pep8-3
- Thomi Richards (community): Approve
- PS Jenkins bot: Approve (continuous-integration)
-
Diff: 389 lines (+77/-58)8 files modifiedautopilot/display/_X11.py (+8/-5)
autopilot/display/__init__.py (+16/-11)
autopilot/display/_upa.py (+1/-0)
autopilot/vis/__init__.py (+1/-1)
autopilot/vis/bus_enumerator.py (+10/-12)
autopilot/vis/main_window.py (+17/-11)
autopilot/vis/objectproperties.py (+16/-10)
autopilot/vis/resources.py (+8/-8)
lp:~elopio/autopilot/fix1201057-pep8-4
- Thomi Richards (community): Needs Fixing
- PS Jenkins bot: Approve (continuous-integration)
-
Diff: 1036 lines (+252/-180)5 files modifiedautopilot/introspection/__init__.py (+91/-55)
autopilot/introspection/backends.py (+28/-23)
autopilot/introspection/dbus.py (+109/-82)
autopilot/introspection/gtk.py (+2/-4)
autopilot/introspection/qt.py (+22/-16)
lp:~elopio/autopilot/fix1201057-pep8-5
- Thomi Richards (community): Approve
- PS Jenkins bot: Approve (continuous-integration)
-
Diff: 1441 lines (+373/-230)10 files modifiedautopilot/tests/functional/test_ap_apps.py (+13/-10)
autopilot/tests/functional/test_application_mixin.py (+26/-13)
autopilot/tests/functional/test_autopilot_functional.py (+256/-164)
autopilot/tests/functional/test_custom_assertions.py (+33/-16)
autopilot/tests/functional/test_dbus_query.py (+2/-1)
autopilot/tests/functional/test_input_stack.py (+5/-2)
autopilot/tests/functional/test_introspection_features.py (+3/-4)
autopilot/tests/functional/test_mouse_emulator.py (+23/-13)
autopilot/tests/functional/test_open_window.py (+2/-1)
autopilot/tests/functional/test_process_emulator.py (+10/-6)
lp:~elopio/autopilot/fix1201057-pep8-6
- Thomi Richards (community): Approve
- PS Jenkins bot: Approve (continuous-integration)
-
Diff: 376 lines (+119/-70)4 files modifiedautopilot/introspection/__init__.py (+1/-1)
autopilot/tests/unit/test_proxy_objects.py (+103/-61)
autopilot/tests/unit/test_start_final_events.py (+1/-1)
autopilot/tests/unit/test_version_utility_fns.py (+14/-7)
lp:~elopio/autopilot/fix1201057-pep8-7
- Thomi Richards (community): Approve
- PS Jenkins bot: Approve (continuous-integration)
-
Diff: 406 lines (+86/-49)8 files modifiedautopilot/tests/unit/test_backend.py (+17/-8)
autopilot/tests/unit/test_command_line_args.py (+4/-2)
autopilot/tests/unit/test_custom_exceptions.py (+9/-4)
autopilot/tests/unit/test_introspection_features.py (+4/-0)
autopilot/tests/unit/test_matchers.py (+11/-6)
autopilot/tests/unit/test_out_of_test_addcleanup.py (+4/-3)
autopilot/tests/unit/test_pick_backend.py (+22/-15)
autopilot/tests/unit/test_platform.py (+15/-11)
lp:~elopio/autopilot/fix1201057-pep8-8
- Thomi Richards (community): Approve
- PS Jenkins bot: Approve (continuous-integration)
-
Diff: 1100 lines (+276/-198)12 files modifiedautopilot/__init__.py (+46/-43)
autopilot/clipboard.py (+2/-2)
autopilot/dbus_handler.py (+1/-1)
autopilot/emulators.py (+5/-7)
autopilot/gestures.py (+6/-4)
autopilot/globals.py (+20/-11)
autopilot/ibus.py (+14/-12)
autopilot/keybindings.py (+29/-19)
autopilot/platform.py (+13/-11)
autopilot/testcase.py (+98/-66)
autopilot/testresult.py (+2/-1)
autopilot/utilities.py (+40/-21)
Changed in autopilot: | |
status: | New → In Progress |
assignee: | nobody → Leo Arias (elopio) |
Changed in autopilot: | |
status: | Fix Committed → In Progress |
Changed in autopilot (Ubuntu): | |
status: | Fix Released → In Progress |
Changed in autopilot: | |
milestone: | none → 1.4 |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
Fix committed into lp:autopilot at revision None, scheduled for release in autopilot, milestone 1.3.2