Prohibit commented code (#396)

pyproject.toml

@@ -43,6 +43,7 @@ flake8-bugbear = "^20.11.1"
 flake8-use-fstring = "^1.1"
 flake8-copyright = "^0.2.2"
 vulture = "^2.3"
+flake8-eradicate = "^1.0.0"
 
 [build-system]
 requires = ["poetry-core>=1.0.0"]

src/pathpicker/output.py

@@ -35,14 +35,11 @@ def exec_composed_command(command: str, line_objs: List[LineMatch]) -> None:
         edit_files(line_objs)
         return
 
-    # decoded_command = command if isinstance(command, str) else command.decode()
-    decoded_command = command
-
     logger.add_event("command_on_num_files", len(line_objs))
-    decoded_command = compose_command(decoded_command, line_objs)
+    command = compose_command(command, line_objs)
     append_alias_expansion()
     append_if_invalid(line_objs)
-    append_friendly_command(decoded_command)
+    append_friendly_command(command)
     append_exit()
 
 

src/tests/lib/curses_api.py

@@ -13,7 +13,7 @@ class CursesForTest(CursesApiBase):
     def __init__(self) -> None:
         self.color_pairs = {}
         self.current_color = (0, 0)
-        # the (0, 0) is hardcoded
+        # The (0, 0) is hardcoded.
         self.color_pairs[0] = self.current_color
 
     def use_default_colors(self) -> None:

src/tests/test_parsing.py

@@ -35,8 +35,7 @@ class ParsingTestCase(NamedTuple):
     ParsingTestCase(".ssh/.gitignore", True, ".ssh/.gitignore"),
     # For now, don't worry about matching the following case perfectly,
     # simply because it's complicated.
-    #    'input': '~/.ssh/known_hosts',
-    #    'match': True,
+    # '~/.ssh/known_hosts', True, '~/.ssh/known_hosts'
     ParsingTestCase(".ssh/known_hosts", True, ".ssh/known_hosts"),
     # Arbitrarily ignore really short dot filenames
     ParsingTestCase(".a", False),