Unabled ruff PYI ruleset for type hint best practices

Author: tleonhardt

cmd2/argparse_custom.py

@@ -264,7 +264,7 @@ def my_completer(self, text, line, begidx, endidx, arg_tokens)
     )
 
 
-def generate_range_error(range_min: int, range_max: Union[int, float]) -> str:
+def generate_range_error(range_min: int, range_max: float) -> str:
     """Generate an error message when the the number of arguments provided is not within the expected range"""
     err_str = "expected "
 

cmd2/table_creator.py

@@ -21,7 +21,6 @@
     Optional,
     Sequence,
     Tuple,
-    Union,
 )
 
 from wcwidth import (  # type: ignore[import]
@@ -69,7 +68,7 @@ def __init__(
         data_horiz_align: HorizontalAlignment = HorizontalAlignment.LEFT,
         data_vert_align: VerticalAlignment = VerticalAlignment.TOP,
         style_data_text: bool = True,
-        max_data_lines: Union[int, float] = constants.INFINITY,
+        max_data_lines: float = constants.INFINITY,
     ) -> None:
         """
         Column initializer
@@ -156,7 +155,7 @@ def __init__(self, cols: Sequence[Column], *, tab_width: int = 4) -> None:
                 col.width = max(1, ansi.widest_line(col.header))
 
     @staticmethod
-    def _wrap_long_word(word: str, max_width: int, max_lines: Union[int, float], is_last_word: bool) -> Tuple[str, int, int]:
+    def _wrap_long_word(word: str, max_width: int, max_lines: float, is_last_word: bool) -> Tuple[str, int, int]:
         """
         Used by _wrap_text() to wrap a long word over multiple lines
 
@@ -220,7 +219,7 @@ def _wrap_long_word(word: str, max_width: int, max_lines: Union[int, float], is_
         return wrapped_buf.getvalue(), total_lines, cur_line_width
 
     @staticmethod
-    def _wrap_text(text: str, max_width: int, max_lines: Union[int, float]) -> str:
+    def _wrap_text(text: str, max_width: int, max_lines: float) -> str:
         """
         Wrap text into lines with a display width no longer than max_width. This function breaks words on whitespace
         boundaries. If a word is longer than the space remaining on a line, then it will start on a new line.

cmd2/utils.py

@@ -709,7 +709,7 @@ def __bool__(self) -> bool:
     def __enter__(self) -> None:
         self.__count += 1
 
-    def __exit__(self, *args: Any) -> None:
+    def __exit__(self, *args: object) -> None:
         self.__count -= 1
         if self.__count < 0:
             raise ValueError("count has gone below 0")

pyproject.toml

@@ -201,7 +201,7 @@ select = [
     # "PLW", # Pylint Warnings
     # "PT", # flake8-pytest-style (warnings about unit test best practices)
     # "PTH",  # flake8-use-pathlib (force use of pathlib instead of os.path)
-    # "PYI", # flake8-pyi (warnings related to type hint best practices)
+    "PYI", # flake8-pyi (warnings related to type hint best practices)
     # "Q",    # flake8-quotes (force double quotes)
     # "RET",  # flake8-return (various warnings related to implicit vs explicit return statements)
     "RSE", # flake8-raise (warn about unnecessary parentheses on raised exceptions)
@@ -238,6 +238,10 @@ per-file-ignores."cmd2/__init__.py" = [
     "F401", # Unused import
 ]
 
+per-file-ignores."cmd2/cmd2.py" = [
+    "PYI024", # Use `typing.NamedTuple` instead of `collections.namedtuple`
+]
+
 per-file-ignores."docs/conf.py" = [
     "F401", # Unused import
 ]
@@ -251,7 +255,7 @@ per-file-ignores."examples/scripts/*.py" = [
 ]
 
 per-file-ignores."examples/unicode_commands.py" = [
-    "PLC2401", # Allow non-ASCII characters in function names
+    "PLC2401", # non-ASCII characters in function names
 ]
 
 per-file-ignores."tests/pyscript/*.py" = [