Code review comment for lp:~3v1n0/ubuntuone-client/sync-menu-use-gappinfo-launch

Revision history for this message
dobey (dobey) wrote :

8 +

E303 too many blank lines (3)

+ def __init__(self, command_line = "", name = "", flags = 0):
+ def __init__(self, desktop_id = ""):
+ def _open_uri(self, uri, timestamp = 0):
+ def _open_control_panel_by_command_line(self, timestamp, args = ''):
+ def open_control_panel(self, menuitem = None, timestamp = 0):
+ def open_ubuntu_one_folder(self, menuitem = None, timestamp = 0):
+ def open_share_file_tab(self, menuitem = None, timestamp = 0):
+ def open_go_to_web(self, menuitem = None, timestamp = 0):
+ def open_web_help(self, menuitem = None, timestamp = 0):
+ def open_get_more_storage(self, menuitem = None, timestamp = 0):

E251 no spaces around keyword / parameter equals

68 + self.context = context
69 +
70 +class FakeDesktopAppInfo(FakeAppInfo):

E302 expected 2 blank lines, found 1

« Back to merge proposal