keyword arguments, $variables, improved macro docs

Author: sezanzeb

.pylintrc

@@ -11,3 +11,6 @@ disable=
     # using """ for comments highlights them in green for me and makes it
     # a great way to separate stuff into multiple sections
     pointless-string-statement
+
+    # https://github.com/psf/black/blob/main/docs/compatible_configs/pylint/pylintrc
+    C0330, C0326

keymapper/gui/window.py

@@ -337,7 +337,7 @@ def check_add_row(self):
         num_maps = len(custom_mapping)
         if num_rows < num_maps or num_rows > num_maps + 1:
             logger.error(
-                "custom_mapping contains %d rows, " "but %d are displayed",
+                "custom_mapping contains %d rows, but %d are displayed",
                 len(custom_mapping),
                 num_rows,
             )

keymapper/injection/injector.py

@@ -68,6 +68,15 @@ def is_in_capabilities(key, capabilities):
     return False
 
 
+def get_udev_name(name, suffix):
+    """Make sure the generated name is not longer than 80 chars."""
+    max_len = 80  # based on error messages
+    remaining_len = max_len - len(DEV_NAME) - len(suffix) - 2
+    middle = name[:remaining_len]
+    name = f"{DEV_NAME} {middle} {suffix}"
+    return name
+
+
 class Injector(multiprocessing.Process):
     """Initializes, starts and stops injections.
 
@@ -310,14 +319,6 @@ async def _msg_listener(self):
                 loop.stop()
                 return
 
-    def get_udev_name(self, name, suffix):
-        """Make sure the generated name is not longer than 80 chars."""
-        max_len = 80  # based on error messages
-        remaining_len = max_len - len(DEV_NAME) - len(suffix) - 2
-        middle = name[:remaining_len]
-        name = f"{DEV_NAME} {middle} {suffix}"
-        return name
-
     def run(self):
         """The injection worker that keeps injecting until terminated.
 
@@ -374,7 +375,7 @@ def run(self):
         # where mapped events go to.
         # See the Context docstring on why this is needed.
         self.context.uinput = evdev.UInput(
-            name=self.get_udev_name(self.group.key, "mapped"),
+            name=get_udev_name(self.group.key, "mapped"),
             phys=DEV_NAME,
             events=self._construct_capabilities(GAMEPAD in self.group.types),
         )
@@ -384,7 +385,7 @@ def run(self):
             # EV_ABS capability, EV_REL won't move the mouse pointer anymore.
             # so don't merge all InputDevices into one UInput device.
             forward_to = evdev.UInput(
-                name=self.get_udev_name(source.name, "forwarded"),
+                name=get_udev_name(source.name, "forwarded"),
                 phys=DEV_NAME,
                 events=self._copy_capabilities(source),
             )