[blivet-gui:master 0/9] Pylint patches
mulhern
amulhern at redhat.com
Wed Jan 14 16:53:33 UTC 2015
_Please_ note warning below.
Initial setup and some fixes. Attached below are remaining pylint errors.
Most are just unused-argument which are probably intentionally unused, due to
GUI interface requirements and probably just need to be suppressed.
A few are no-member; maybe set_margin_start should be set_margin_left, but
I am not sure.
It is customary to put this stuff in tests/pylint subdirectory, so I did.
I had to remove tests entry from .gitignore. Please make sure not to kill
your local private tests directory if such a thing exists, by applying
these patches. Thanks.
************* Module blivetgui/actions_toolbar.py
blivetgui/actions_toolbar.py:152: [W0613(unused-argument), ActionsToolbar.on_delete_clicked] Unused argument 'button'
blivetgui/actions_toolbar.py:158: [W0613(unused-argument), ActionsToolbar.on_add_clicked] Unused argument 'button'
blivetgui/actions_toolbar.py:164: [W0613(unused-argument), ActionsToolbar.on_edit_clicked] Unused argument 'button'
blivetgui/actions_toolbar.py:170: [W0613(unused-argument), ActionsToolbar.on_umount_clicked] Unused argument 'button'
blivetgui/actions_toolbar.py:176: [W0613(unused-argument), ActionsToolbar.on_decrypt_clicked] Unused argument 'button'
blivetgui/actions_toolbar.py:182: [W0613(unused-argument), ActionsToolbar.on_apply_clicked] Unused argument 'button'
blivetgui/actions_toolbar.py:188: [W0613(unused-argument), ActionsToolbar.on_clear_clicked] Unused argument 'button'
blivetgui/actions_toolbar.py:194: [W0613(unused-argument), ActionsToolbar.on_undo_clicked] Unused argument 'button'
************* Module blivetgui/dialogs/edit_dialog.py
blivetgui/dialogs/edit_dialog.py:164: [W0613(unused-argument), PartitionEditDialog.on_format_changed] Unused argument 'event'
blivetgui/dialogs/edit_dialog.py:382: [W0613(unused-argument), LVMEditDialog.on_cell_toggled] Unused argument 'toggle'
blivetgui/dialogs/edit_dialog.py:385: [W0613(unused-argument), LVMEditDialog.on_cell_radio_toggled] Unused argument 'toggle'
************* Module blivetgui/list_partitions.py
blivetgui/list_partitions.py:77: [E1101(no-member), ListPartitions.__init__] Instance of 'Label' has no 'set_margin_start' member
blivetgui/list_partitions.py:78: [E1101(no-member), ListPartitions.__init__] Instance of 'Label' has no 'set_margin_end' member
blivetgui/list_partitions.py:863: [W0613(unused-argument), ListPartitions.quit] Unused argument 'widget'
blivetgui/list_partitions.py:863: [W0613(unused-argument), ListPartitions.quit] Unused argument 'event'
************* Module blivetgui/dialogs/other_dialogs.py
blivetgui/dialogs/other_dialogs.py:187: [W0613(unused-argument), KickstartSelectDevicesDialog.on_cell_toggled] Unused argument 'event'
blivetgui/dialogs/other_dialogs.py:214: [W0613(unused-argument), KickstartSelectDevicesDialog.on_boot_changed] Unused argument 'event'
************* Module blivetgui/dialogs/add_dialog.py
blivetgui/dialogs/add_dialog.py:264: [W0613(unused-argument), SizeChooserArea.get_size_precision] Unused argument 'unit'
blivetgui/dialogs/add_dialog.py:683: [W0613(unused-argument), AddDialog.on_raid_type_changed] Unused argument 'event'
blivetgui/dialogs/add_dialog.py:807: [W0613(unused-argument), AddDialog.on_cell_toggled] Unused argument 'event'
blivetgui/dialogs/add_dialog.py:983: [W0613(unused-argument), AddDialog.on_md_type_changed] Unused argument 'event'
blivetgui/dialogs/add_dialog.py:1129: [W0613(unused-argument), AddDialog.on_devices_combo_changed] Unused argument 'event'
blivetgui/dialogs/add_dialog.py:1253: [W0613(unused-argument), AddDialog.on_ok_clicked] Unused argument 'event'
************* Module blivetgui/devicevisualization/device_canvas.py
blivetgui/devicevisualization/device_canvas.py:390: [W0613(unused-argument), DeviceCanvas.button_press_event] Unused argument 'darea'
************* Module blivetgui/actions_menu.py
blivetgui/actions_menu.py:129: [W0613(unused-argument), ActionsMenu.on_add_item] Unused argument 'event'
blivetgui/actions_menu.py:134: [W0613(unused-argument), ActionsMenu.on_delete_item] Unused argument 'event'
blivetgui/actions_menu.py:139: [W0613(unused-argument), ActionsMenu.on_edit_item] Unused argument 'event'
blivetgui/actions_menu.py:144: [W0613(unused-argument), ActionsMenu.on_umount_item] Unused argument 'event'
blivetgui/actions_menu.py:149: [W0613(unused-argument), ActionsMenu.on_decrypt_item] Unused argument 'event'
************* Module examples/embeded.py
examples/embeded.py:58: [W0613(unused-argument), SocketWindow.plugged_event] Unused argument 'widget'
************* Module blivetgui/main_menu.py
blivetgui/main_menu.py:256: [W0613(unused-argument), MainMenu.on_about_item] Unused argument 'args'
blivetgui/main_menu.py:262: [W0613(unused-argument), MainMenu.on_help_item] Unused argument 'args'
blivetgui/main_menu.py:295: [W0613(unused-argument), MainMenu.on_undo_item] Unused argument 'args'
blivetgui/main_menu.py:301: [W0613(unused-argument), MainMenu.on_clear_item] Unused argument 'args'
blivetgui/main_menu.py:307: [W0613(unused-argument), MainMenu.on_apply_item] Unused argument 'args'
blivetgui/main_menu.py:313: [W0613(unused-argument), MainMenu.on_add_item] Unused argument 'args'
blivetgui/main_menu.py:319: [W0613(unused-argument), MainMenu.on_delete_item] Unused argument 'args'
blivetgui/main_menu.py:325: [W0613(unused-argument), MainMenu.on_edit_item] Unused argument 'args'
blivetgui/main_menu.py:331: [W0613(unused-argument), MainMenu.on_umount_item] Unused argument 'args'
blivetgui/main_menu.py:337: [W0613(unused-argument), MainMenu.on_decrypt_item] Unused argument 'args'
blivetgui/main_menu.py:343: [W0613(unused-argument), MainMenu.on_quit_item] Unused argument 'args'
blivetgui/main_menu.py:349: [W0613(unused-argument), MainMenu.on_reload_item] Unused argument 'args'
************* Module blivetgui/processing_window.py
blivetgui/processing_window.py:112: [W0613(unused-argument), ProcessingActions.on_timeout] Unused argument 'user_data'
mulhern (9):
Initial pylint setup.
Fix bad indentation.
Do not use builtin name format as parameter name.
Do not use wildcard import.
Move % operator outside translation.
Omit needless imports.
Omit or hide unused variables and computations.
Change relative to absolute imports.
Suppress broad-except pylint errors.
.gitignore | 1 -
blivet-gui | 2 +-
blivetgui/devicevisualization/device_canvas.py | 2 +-
blivetgui/dialogs/edit_dialog.py | 6 +-
blivetgui/list_partitions.py | 2 +-
blivetgui/main.py | 8 +-
blivetgui/processing_window.py | 4 +-
blivetgui/udisks_loop.py | 3 -
blivetgui/utils.py | 16 +-
examples/embeded.py | 4 +-
tests/lib/testlib.sh | 37 ++++
tests/pylint/intl.py | 63 +++++++
tests/pylint/pointless-override.py | 252 +++++++++++++++++++++++++
tests/pylint/pylint-false-positives | 0
tests/pylint/pylint-one.sh | 33 ++++
tests/pylint/runpylint.sh | 125 ++++++++++++
tests/testenv.sh | 13 ++
17 files changed, 544 insertions(+), 27 deletions(-)
create mode 100644 tests/lib/testlib.sh
create mode 100644 tests/pylint/intl.py
create mode 100644 tests/pylint/pointless-override.py
create mode 100644 tests/pylint/pylint-false-positives
create mode 100755 tests/pylint/pylint-one.sh
create mode 100755 tests/pylint/runpylint.sh
create mode 100644 tests/testenv.sh
--
1.9.3
More information about the anaconda-patches
mailing list