lint and format

Author: Tony Crisci

.flake8

@@ -0,0 +1,10 @@
+[flake8]
+ignore=
+    E501
+    E126
+    E402
+    F722
+    W503
+
+per-file-ignores=
+    */__init__.py:F401

.style.yapf

@@ -0,0 +1,2 @@
+[style]
+column_limit = 100

examples/app-on-ws-init.py

@@ -13,9 +13,14 @@
     workspace 6.
     """)
 
-parser.add_argument('--workspace', metavar='WS_NAME', nargs='+', required=True,
+parser.add_argument('--workspace',
+                    metavar='WS_NAME',
+                    nargs='+',
+                    required=True,
                     help='The name of the workspaces to run the command on')
-parser.add_argument('--command', metavar='CMD', required=True,
+parser.add_argument('--command',
+                    metavar='CMD',
+                    required=True,
                     help='The command to run on the newly initted workspace')
 
 args = parser.parse_args()
@@ -25,6 +30,7 @@ def on_workspace(i3, e):
     if e.current.props.name in args.workspace and not len(e.current.leaves()):
         i3.command('exec {}'.format(args.command))
 
+
 i3.on('workspace::focus', on_workspace)
 
 i3.main()

examples/asyncio-i3status-wrapper.py

@@ -2,7 +2,7 @@
 # This module is an example of how to use i3ipc with asyncio event loop. It
 # implements an i3status wrapper that handles a special keybinding to switch
 # keyboard layout, while also displaying current layout in i3bar.
-# 
+#
 # The keyboard layout switcher can be activated by adding something like this
 # to i3 config:
 #
@@ -30,7 +30,7 @@
     # config file. This is only done for demonstration purposes.
     import setproctitle
     setproctitle.setproctitle('i3bar')
-except ImportError as e:
+except ImportError:
     # Configure i3status by explicitly setting "i3bar" as output_format
     configure_i3_status = True
 
@@ -67,9 +67,7 @@ def __init__(self):
         self.first_write = True
         self.layouts = ['us', 'us intl']
         self.current_layout = -1
-        self.command_handlers = {
-            'switch_layout': lambda: self.switch_layout()
-        }
+        self.command_handlers = {'switch_layout': lambda: self.switch_layout()}
         # perform a switch now, which will force the keyboard layout to be
         # shown before other data
         self.switch_layout()
@@ -78,11 +76,7 @@ def switch_layout(self):
         self.current_layout = (self.current_layout + 1) % len(self.layouts)
         new_layout = self.layouts[self.current_layout]
         subprocess.call('setxkbmap {}'.format(new_layout), shell=True)
-        self.update([{
-            'name': 'keyboard_layout',
-            'markup': 'none',
-            'full_text': new_layout
-        }])
+        self.update([{'name': 'keyboard_layout', 'markup': 'none', 'full_text': new_layout}])
 
     def dispatch_command(self, command):
         c = command.split(' ')
@@ -101,9 +95,10 @@ def update(self, new_status):
     def repaint(self):
         template = '{}' if self.first_write else ',{}'
         self.first_write = False
-        sys.stdout.write(template.format(
-            json.dumps([item for item in self.current_status.values()
-                        if item], separators=(',', ':'))))
+        sys.stdout.write(
+            template.format(
+                json.dumps([item for item in self.current_status.values() if item],
+                           separators=(',', ':'))))
         sys.stdout.write('\n')
         sys.stdout.flush()
 
@@ -118,27 +113,23 @@ def handle_i3status_payload(line):
             cfg_file = tempfile.NamedTemporaryFile(mode='w+b')
             cfg_file.write(I3STATUS_CFG.encode('utf8'))
             cfg_file.flush()
-            create = asyncio.create_subprocess_exec(
-                'i3status', '-c', cfg_file.name,
-                stdout=asyncio.subprocess.PIPE)
+            create = asyncio.create_subprocess_exec('i3status',
+                                                    '-c',
+                                                    cfg_file.name,
+                                                    stdout=asyncio.subprocess.PIPE)
         else:
-            create = asyncio.create_subprocess_exec(
-                'i3status', stdout=asyncio.subprocess.PIPE)
+            create = asyncio.create_subprocess_exec('i3status', stdout=asyncio.subprocess.PIPE)
         i3status = yield from create
         # forward first line, version information
-        sys.stdout.write(
-            (yield from i3status.stdout.readline()).decode('utf8'))
+        sys.stdout.write((yield from i3status.stdout.readline()).decode('utf8'))
         # forward second line, an opening list bracket (no idea why this
         # exists)
-        sys.stdout.write(
-            (yield from i3status.stdout.readline()).decode('utf8'))
+        sys.stdout.write((yield from i3status.stdout.readline()).decode('utf8'))
         # third line is a json payload
-        handle_i3status_payload(
-            (yield from i3status.stdout.readline()).decode('utf8'))
+        handle_i3status_payload((yield from i3status.stdout.readline()).decode('utf8'))
         while True:
             # all subsequent lines are json payload with a leading comma
-            handle_i3status_payload(
-                (yield from i3status.stdout.readline()).decode('utf8')[1:])
+            handle_i3status_payload((yield from i3status.stdout.readline()).decode('utf8')[1:])
 
 
 status = Status()

examples/command-on-exit.py

@@ -5,14 +5,16 @@
 # https://faq.i3wm.org/question/3468/run-a-command-when-i3-exits/
 
 # This is the command to run
-COMMAND = [ 'echo', 'hello, world' ]
+COMMAND = ['echo', 'hello, world']
 
 from subprocess import Popen
 import i3ipc
 
+
 def on_shutdown(i3):
     Popen(COMMAND)
 
+
 i3 = i3ipc.Connection()
 
 i3.on('ipc_shutdown', on_shutdown)

examples/disable-standby-fs.py

@@ -7,17 +7,18 @@
 i3 = i3ipc.Connection()
 
 parser = ArgumentParser(prog='disable-standby-fs',
-        description='''
+                        description='''
         Disable standby (dpms) and screensaver when a window becomes fullscreen
         or exits fullscreen-mode. Requires `xorg-xset`.
         ''')
 
 args = parser.parse_args()
 
+
 def find_fullscreen(con):
     # XXX remove me when this method is available on the con in a release
-    return [c for c in con.descendents()
-            if c.type == 'con' and c.fullscreen_mode]
+    return [c for c in con.descendents() if c.type == 'con' and c.fullscreen_mode]
+
 
 def set_dpms(state):
     if state:
@@ -29,13 +30,16 @@ def set_dpms(state):
         call(['xset', 's', 'off'])
         call(['xset', '-dpms'])
 
+
 def on_fullscreen_mode(i3, e):
     set_dpms(not len(find_fullscreen(i3.get_tree())))
 
+
 def on_window_close(i3, e):
     if not len(find_fullscreen(i3.get_tree())):
         set_dpms(True)
 
+
 i3.on('window::fullscreen_mode', on_fullscreen_mode)
 i3.on('window::close', on_window_close)
 

examples/focus-last.py

@@ -12,12 +12,10 @@
 
 
 class FocusWatcher:
-
     def __init__(self):
         self.i3 = i3ipc.Connection()
         self.i3.on('window::focus', self.on_window_focus)
-        self.listening_socket = socket.socket(socket.AF_UNIX,
-            socket.SOCK_STREAM)
+        self.listening_socket = socket.socket(socket.AF_UNIX, socket.SOCK_STREAM)
         if os.path.exists(SOCKET_FILE):
             os.remove(SOCKET_FILE)
         self.listening_socket.bind(SOCKET_FILE)
@@ -73,18 +71,22 @@ def run(self):
         for t in (t_i3, t_server):
             t.start()
 
+
 if __name__ == '__main__':
     parser = ArgumentParser(prog='focus-last.py',
-        description='''
+                            description='''
         Focus last focused window.
 
         This script should be launch from the .xsessionrc without argument.
 
         Then you can bind this script with the `--switch` option to one of your
         i3 keybinding.
         ''')
-    parser.add_argument('--switch', dest='switch', action='store_true',
-        help='Switch to the previous window', default=False)
+    parser.add_argument('--switch',
+                        dest='switch',
+                        action='store_true',
+                        help='Switch to the previous window',
+                        default=False)
     args = parser.parse_args()
 
     if not args.switch:

examples/focus-next-visible.py

@@ -1,5 +1,4 @@
 #!/usr/bin/env python3
-
 """
 focus-next-visible.py - cycles input focus between visible windows on workspace
 
@@ -26,19 +25,17 @@
 
 
 def get_windows_on_ws(conn):
-    return filter(lambda x: x.window,
-                  conn.get_tree().find_focused().workspace().descendents())
+    return filter(lambda x: x.window, conn.get_tree().find_focused().workspace().descendents())
 
 
 def find_visible_windows(windows_on_workspace):
     visible_windows = []
     for w in windows_on_workspace:
-     
+
         try:
             xprop = check_output(['xprop', '-id', str(w.window)]).decode()
         except FileNotFoundError:
-            raise SystemExit("The `xprop` utility is not found!"
-                             " Please install it and retry.")
+            raise SystemExit("The `xprop` utility is not found!" " Please install it and retry.")
 
         if '_NET_WM_STATE_HIDDEN' not in xprop:
             visible_windows.append(w)

examples/focused-windows.py

@@ -5,30 +5,30 @@
 
 i3 = i3ipc.Connection()
 
+
 def focused_windows():
-  tree = i3.get_tree()
+    tree = i3.get_tree()
 
-  workspaces = tree.workspaces()
-  for workspace in workspaces:
-    container = workspace
+    workspaces = tree.workspaces()
+    for workspace in workspaces:
+        container = workspace
 
-    while container:
-      if not hasattr(container, 'focus') \
-          or not container.focus:
-        break
+        while container:
+            if not hasattr(container, 'focus') or not container.focus:
+                break
 
-      container_id = container.focus[0]
-      container = container.find_by_id(container_id)
+            container_id = container.focus[0]
+            container = container.find_by_id(container_id)
 
-    if container:
-      coname = container.name
-      wsname = workspace.name
+        if container:
+            coname = container.name
+            wsname = workspace.name
 
-      print('WS', wsname +':', coname)
+            print('WS', wsname + ':', coname)
 
 
 if __name__ == '__main__':
-  parser = ArgumentParser(description = 'Print the names of the focused window of each workspace.')
-  parser.parse_args()
+    parser = ArgumentParser(description='Print the names of the focused window of each workspace.')
+    parser.parse_args()
 
-  focused_windows()
+    focused_windows()

examples/i3-cmd.py

@@ -2,7 +2,7 @@
 
 import i3ipc
 from argparse import ArgumentParser
-from subprocess import check_output, Popen, PIPE, CalledProcessError
+from subprocess import check_output, Popen, CalledProcessError
 from sys import exit
 from os.path import basename
 
@@ -42,8 +42,8 @@
 cmd = ''
 
 try:
-    cmd = check_output([ args.menu ] + menu_args,
-            input=bytes('\n'.join(history), 'UTF-8')).decode('UTF-8').strip()
+    cmd = check_output([args.menu] + menu_args, input=bytes('\n'.join(history),
+                                                            'UTF-8')).decode('UTF-8').strip()
 except CalledProcessError as e:
     exit(e.returncode)