lint 10/10!

Author: jquast

telnetlib3/server_pty_shell.py

@@ -104,6 +104,7 @@ def start(self) -> None:
 
         :raises PTYSpawnError: If the child process fails to exec.
         """
+        # pylint: disable=import-outside-toplevel
         # std imports
         import pty
         import fcntl
@@ -223,6 +224,7 @@ def _setup_child(
     ) -> None:
         """Child process setup before exec."""
         # Note: pty.fork() already calls setsid() for the child, so we don't need to
+        # pylint: disable=import-outside-toplevel
         # std imports
         import fcntl
         import termios
@@ -263,6 +265,7 @@ def _setup_child(
 
     def _setup_parent(self) -> None:
         """Parent process setup after fork."""
+        # pylint: disable=import-outside-toplevel
         # std imports
         import fcntl
 
@@ -294,6 +297,7 @@ def _fire_naws_update(self) -> None:
 
     def _set_window_size(self, rows: int, cols: int) -> None:
         """Set PTY window size and send SIGWINCH to child."""
+        # pylint: disable=import-outside-toplevel
         # std imports
         import fcntl
         import signal
@@ -310,6 +314,7 @@ def _set_window_size(self, rows: int, cols: int) -> None:
 
     async def run(self) -> None:
         """Bridge loop between telnet and PTY."""
+        # pylint: disable=import-outside-toplevel
         # std imports
         import errno
 
@@ -534,6 +539,7 @@ def _terminate(self, force: bool = False) -> bool:
         :param force: If True, use SIGKILL as last resort.
         :returns: True if child was terminated, False otherwise.
         """
+        # pylint: disable=import-outside-toplevel
         # std imports
         import signal
 

telnetlib3/slc.py

@@ -389,13 +389,13 @@ def description_table(self) -> List[str]:
         mrk_cont = "(...)"
 
         def continuing() -> bool:
-            return bool(len(result) and result[-1] == mrk_cont)
+            return bool(result and result[-1] == mrk_cont)
 
         def is_last(mask: int) -> bool:
             return mask == len(self.value) - 1
 
         def same_as_last(row: str) -> bool:
-            return bool(len(result) and result[-1].endswith(row.split()[-1]))
+            return bool(result and result[-1].endswith(row.split()[-1]))
 
         for mask, byte in enumerate(self.value):
             if byte == 0: