Improve type hints.

diff --git a/src/pyutils/bootstrap.py b/src/pyutils/bootstrap.py
index f096add5bac15c892d97bf3ff7cd02d540a1ff24..7ef7ef13963a2a8cdb92c86c2657dca4826d9e92 100644 (file)
--- a/src/pyutils/bootstrap.py
+++ b/src/pyutils/bootstrap.py
@@ -45,6 +45,7 @@ import os
 import sys
 import uuid
 from inspect import stack
+from typing import NoReturn
 
 from pyutils import config, logging_utils
 from pyutils.argparse_utils import ActionNoYes
@@ -178,7 +179,7 @@ class ImportInterceptor(importlib.abc.MetaPathFinder):
     def should_ignore_filename(filename: str) -> bool:
         return "importlib" in filename or "six.py" in filename
 
-    def find_module(self, fullname, path):
+    def find_module(self, fullname, path) -> NoReturn:
         raise Exception(
             "This method has been deprecated since Python 3.4, please upgrade."
         )

diff --git a/src/pyutils/config.py b/src/pyutils/config.py
index 9655f251f2e30b47f7e17702be81fbf31246c07b..456be3e96fbc2e1400ce952f18fcdb67f81d479a 100644 (file)
--- a/src/pyutils/config.py
+++ b/src/pyutils/config.py
@@ -410,7 +410,7 @@ class Config:
         """
         return in_str.lower() in {"true", "1", "yes", "y", "t", "on"}
 
-    def _process_dynamic_args(self, event):
+    def _process_dynamic_args(self, event) -> None:
         """Invoked as a callback when a zk-based config changed."""
 
         if not self.zk:

diff --git a/src/pyutils/decorator_utils.py b/src/pyutils/decorator_utils.py
index 1a429cfe0e35135c75823015e6e815a072716622..f0856975930ec2b76c8df95624fbe2ef3cae8a53 100644 (file)
--- a/src/pyutils/decorator_utils.py
+++ b/src/pyutils/decorator_utils.py
@@ -18,7 +18,7 @@ import threading
 import time
 import traceback
 import warnings
-from typing import Any, Callable, List, Optional, Union
+from typing import Any, Callable, List, NoReturn, Optional, Union
 
 # This module is commonly used by others in here and should avoid
 # taking any unnecessary dependencies back on them.
@@ -706,7 +706,7 @@ def thunkify(func):
 # to be bound by the terms and conditions of this License Agreement.
 
 
-def _raise_exception(exception, error_message: Optional[str]):
+def _raise_exception(exception, error_message: Optional[str]) -> NoReturn:
     """Internal.  Raise a deferred exception"""
     if error_message is None:
         raise Exception(exception)