adjust line length to maintain length < 90

Author: chrissimpkins

lib/fdiff/__main__.py

@@ -124,14 +124,16 @@ def run(argv):
         # head and tail are not supported when external diff tool is called
         if args.head or args.tail:
             sys.stderr.write(
-                f"[ERROR] The head and tail options are not supported with external diff executable calls.{os.linesep}"
+                f"[ERROR] The head and tail options are not supported with external "
+                f"diff executable calls.{os.linesep}"
             )
             sys.exit(1)
 
         # lines of context filter is not supported when external diff tool is called
         if args.lines != 3:
             sys.stderr.write(
-                f"[ERROR] The lines option is not supported with external diff executable calls.{os.linesep}"
+                f"[ERROR] The lines option is not supported with external diff "
+                f"executable calls.{os.linesep}"
             )
             sys.exit(1)
 

lib/fdiff/diff.py

@@ -39,7 +39,8 @@ def _async_fetch_files(dirpath, urls):
         elif task.result().http_status != 200:
             # handle non-200 HTTP response status codes + file write fails
             raise AIOError(
-                f"failed to pull '{task.result().url}' with HTTP status code {task.result().http_status}"
+                f"failed to pull '{task.result().url}' with HTTP status "
+                f"code {task.result().http_status}"
             )
 
 
@@ -188,7 +189,8 @@ def u_diff(
     that is not included in filepath_a OR filepath_b
     :raises: fdiff.exceptions.AIOError if exception raised during execution of async I/O
              GET request for URL or file write
-    :raises: fdiff.exceptions.AIOError if GET request to URL returned non-200 response status code"""
+    :raises: fdiff.exceptions.AIOError if GET request to URL returned non-200 response
+    status code"""
     with tempfile.TemporaryDirectory() as tmpdirpath:
         # define the file paths with either local file requests
         # or HTTP GET requests of remote files based on the command line request
@@ -236,23 +238,24 @@ def external_diff(
     use_multiprocess=True,
 ):
     """Performs a unified diff on a TTX serialized data format dump of font binary data using
-        an external diff executable that is requested by the caller via `command`
-
-        command: (string) command line executable string and arguments to define execution
-        filepath_a: (string) pre-file local file path or URL path
-        filepath_b: (string) post-file local file path or URL path
-        include_tables: (list of str) Python list of OpenType tables to include in the diff
-        exclude_tables: (list of str) Python list of OpentType tables to exclude from the diff
-        use_multiprocess: (bool) use multi-processor optimizations (default=True)
-
-        include_tables and exclude_tables are mutually exclusive arguments.  Only one should
-        be defined
-
-        :returns: Generator of ordered diff line strings that include newline line endings
-        :raises: KeyError if include_tables or exclude_tables includes a mis-specified table
-        that is not included in filepath_a OR filepath_b
-        :raises: IOError if exception raised during execution of `command` on TTX files
-        :raises: fdiff.exceptions.AIOError if GET request to URL returned non-200 response status code"""
+    an external diff executable that is requested by the caller via `command`
+
+    command: (string) command line executable string and arguments to define execution
+    filepath_a: (string) pre-file local file path or URL path
+    filepath_b: (string) post-file local file path or URL path
+    include_tables: (list of str) Python list of OpenType tables to include in the diff
+    exclude_tables: (list of str) Python list of OpentType tables to exclude from the diff
+    use_multiprocess: (bool) use multi-processor optimizations (default=True)
+
+    include_tables and exclude_tables are mutually exclusive arguments.  Only one should
+    be defined
+
+    :returns: Generator of ordered diff line strings that include newline line endings
+    :raises: KeyError if include_tables or exclude_tables includes a mis-specified table
+    that is not included in filepath_a OR filepath_b
+    :raises: IOError if exception raised during execution of `command` on TTX files
+    :raises: fdiff.exceptions.AIOError if GET request to URL returned non-200 response
+    status code"""
     with tempfile.TemporaryDirectory() as tmpdirpath:
         # define the file paths with either local file requests
         # or HTTP GET requests of remote files based on the command line request

lib/fdiff/remote.py

@@ -20,7 +20,8 @@ def _get_filepath_from_url(url, dirpath):
 
 
 async def async_fetch(session, url):
-    """Asynchronous I/O HTTP GET request with a ClientSession instantiated from the aiohttp library."""
+    """Asynchronous I/O HTTP GET request with a ClientSession instantiated
+    from the aiohttp library."""
     async with session.get(url) as response:
         status = response.status
         if status != 200:
@@ -31,8 +32,9 @@ async def async_fetch(session, url):
 
 
 async def async_fetch_and_write(session, url, dirpath):
-    """Asynchronous I/O HTTP GET request with a ClientSession instantiated from the aiohttp library, followed
-    by an asynchronous I/O file write of the binary to disk with the aiofiles library.
+    """Asynchronous I/O HTTP GET request with a ClientSession instantiated
+    from the aiohttp library, followed by an asynchronous I/O file write of
+    the binary to disk with the aiofiles library.
 
     :returns `FWRes` namedtuple with url, filepath, http_status, write_success fields"""
     FWResponse = namedtuple(
@@ -56,7 +58,8 @@ async def create_async_get_request_session_and_run(urls, dirpath):
     """Creates an aiohttp library ClientSession and performs asynchronous GET requests +
     binary file writes with the binary response from the GET request.
 
-    :returns list of asyncio Tasks that include `FWRes` namedtuple instances (defined in async_fetch_and_write)"""
+    :returns list of asyncio Tasks that include `FWRes` namedtuple instances
+    (defined in async_fetch_and_write)"""
     async with aiohttp.ClientSession() as session:
         tasks = []
         for url in urls:

lib/fdiff/utils.py

@@ -20,9 +20,10 @@ def get_file_modtime(path):
 
 
 def get_tables_argument_list(table_string):
-    """Converts a comma separated OpenType table string into a Python list or return None if
-    the table_string was not defined (i.e., it was not included in an option on the command line).
-    Tables that are composed of three characters must be right padded with a space."""
+    """Converts a comma separated OpenType table string into a Python list or
+    return None if the table_string was not defined (i.e., it was not included
+    in an option on the command line). Tables that are composed of three
+    characters must be right padded with a space."""
     if table_string is None:
         return None
     else: