Enable ruff ARG ruleset to warn about unused arguments (#1425)

Author: tleonhardt

cmd2/argparse_custom.py

@@ -280,7 +280,7 @@ class CompletionItem(str):  # noqa: SLOT000
     See header of this file for more information
     """
 
-    def __new__(cls, value: object, *args: Any, **kwargs: Any) -> 'CompletionItem':
+    def __new__(cls, value: object, *_args: Any, **_kwargs: Any) -> 'CompletionItem':
         return super().__new__(cls, value)
 
     def __init__(self, value: object, description: str = '', *args: Any) -> None:
@@ -914,7 +914,7 @@ def _ArgumentParser_set_ap_completer_type(self: argparse.ArgumentParser, ap_comp
 ############################################################################################################
 # Patch ArgumentParser._check_value to support CompletionItems as choices
 ############################################################################################################
-def _ArgumentParser_check_value(self: argparse.ArgumentParser, action: argparse.Action, value: Any) -> None:  # noqa: N802
+def _ArgumentParser_check_value(_self: argparse.ArgumentParser, action: argparse.Action, value: Any) -> None:  # noqa: N802
     """Custom override of ArgumentParser._check_value that supports CompletionItems as choices.
     When evaluating choices, input is compared to CompletionItem.orig_value instead of the
     CompletionItem instance.

cmd2/cmd2.py

@@ -1111,7 +1111,7 @@ def remove_settable(self, name: str) -> None:
     def build_settables(self) -> None:
         """Create the dictionary of user-settable parameters."""
 
-        def get_allow_style_choices(cli_self: Cmd) -> list[str]:
+        def get_allow_style_choices(_cli_self: Cmd) -> list[str]:
             """Used to tab complete allow_style values."""
             return [val.name.lower() for val in ansi.AllowStyle]
 
@@ -1419,9 +1419,9 @@ def tokens_for_completion(self, line: str, begidx: int, endidx: int) -> tuple[li
     def basic_complete(
         self,
         text: str,
-        line: str,
-        begidx: int,
-        endidx: int,
+        line: str,  # noqa: ARG002
+        begidx: int,  # noqa: ARG002
+        endidx: int,  # noqa: ARG002
         match_against: Iterable[str],
     ) -> list[str]:
         """Basic tab completion function that matches against a list of strings without considering line contents
@@ -1602,7 +1602,13 @@ def index_based_complete(
         return matches
 
     def path_complete(
-        self, text: str, line: str, begidx: int, endidx: int, *, path_filter: Optional[Callable[[str], bool]] = None
+        self,
+        text: str,
+        line: str,
+        begidx: int,  # noqa: ARG002
+        endidx: int,
+        *,
+        path_filter: Optional[Callable[[str], bool]] = None,
     ) -> list[str]:
         """Performs completion of local file system paths.
 
@@ -2346,7 +2352,7 @@ def get_help_topics(self) -> list[str]:
         # Filter out hidden and disabled commands
         return [topic for topic in all_topics if topic not in self.hidden_commands and topic not in self.disabled_commands]
 
-    def sigint_handler(self, signum: int, _: Optional[FrameType]) -> None:
+    def sigint_handler(self, signum: int, _: Optional[FrameType]) -> None:  # noqa: ARG002
         """Signal handler for SIGINTs which typically come from Ctrl-C events.
 
         If you need custom SIGINT behavior, then override this method.
@@ -2402,7 +2408,7 @@ def precmd(self, statement: Union[Statement, str]) -> Statement:
         """
         return Statement(statement) if not isinstance(statement, Statement) else statement
 
-    def postcmd(self, stop: bool, statement: Union[Statement, str]) -> bool:
+    def postcmd(self, stop: bool, statement: Union[Statement, str]) -> bool:  # noqa: ARG002
         """Hook method executed just after a command is executed by
         [cmd2.Cmd.onecmd][].
 
@@ -3087,7 +3093,7 @@ def configure_readline() -> None:
                 # Disable completion
                 if completion_mode == utils.CompletionMode.NONE:
 
-                    def complete_none(text: str, state: int) -> Optional[str]:  # pragma: no cover
+                    def complete_none(text: str, state: int) -> Optional[str]:  # pragma: no cover  # noqa: ARG001
                         return None
 
                     complete_func = complete_none
@@ -3773,7 +3779,7 @@ def do_help(self, args: argparse.Namespace) -> None:
                 self.perror(err_msg, apply_style=False)
                 self.last_result = False
 
-    def print_topics(self, header: str, cmds: Optional[list[str]], cmdlen: int, maxcol: int) -> None:
+    def print_topics(self, header: str, cmds: Optional[list[str]], cmdlen: int, maxcol: int) -> None:  # noqa: ARG002
         """Print groups of commands and topics in columns and an optional header
         Override of cmd's print_topics() to handle headers with newlines, ANSI style sequences, and wide characters.
 
@@ -5379,7 +5385,7 @@ def disable_command(self, command: str, message_to_print: str) -> None:
         setattr(self, help_func_name, new_func)
 
         # Set the completer to a function that returns a blank list
-        setattr(self, completer_func_name, lambda *args, **kwargs: [])
+        setattr(self, completer_func_name, lambda *_args, **_kwargs: [])
 
     def disable_category(self, category: str, message_to_print: str) -> None:
         """Disable an entire category of commands.

cmd2/parsing.py

@@ -145,7 +145,7 @@ class Statement(str):  # type: ignore[override]  # noqa: SLOT000
     # Used in JSON dictionaries
     _args_field = 'args'
 
-    def __new__(cls, value: object, *pos_args: Any, **kw_args: Any) -> 'Statement':
+    def __new__(cls, value: object, *_pos_args: Any, **_kw_args: Any) -> 'Statement':
         """Create a new instance of Statement.
 
         We must override __new__ because we are subclassing `str` which is

examples/basic_completion.py

@@ -82,7 +82,7 @@ def do_raise_error(self, statement: cmd2.Statement) -> None:
         """Demonstrates effect of raising CompletionError."""
         self.poutput(f"Args: {statement.args}")
 
-    def complete_raise_error(self, text, line, begidx, endidx) -> list[str]:
+    def complete_raise_error(self, _text, _line, _begidx, _endidx) -> list[str]:
         """CompletionErrors can be raised if an error occurs while tab completing.
 
         Example use cases

examples/default_categories.py

@@ -67,7 +67,7 @@ class ExampleApp(cmd2.Cmd):
     def __init__(self) -> None:
         super().__init__()
 
-    def do_something(self, arg) -> None:
+    def do_something(self, _arg) -> None:
         self.poutput('this is the something command')
 
 

examples/dynamic_commands.py

@@ -34,7 +34,7 @@ def __init__(self) -> None:
 
         super().__init__(include_ipy=True)
 
-    def send_text(self, args: cmd2.Statement, *, text: str) -> None:
+    def send_text(self, _args: cmd2.Statement, *, text: str) -> None:
         """Simulate sending text to a server and printing the response."""
         self.poutput(text.capitalize())
 

examples/environment.py

@@ -16,7 +16,7 @@ def __init__(self) -> None:
         )
         self.add_settable(cmd2.Settable('sunny', bool, 'Is it sunny outside?', self))
 
-    def do_sunbathe(self, arg) -> None:
+    def do_sunbathe(self, _arg) -> None:
         """Attempt to sunbathe."""
         if self.degrees_c < 20:
             result = f"It's {self.degrees_c} C - are you a penguin?"
@@ -26,7 +26,7 @@ def do_sunbathe(self, arg) -> None:
             result = 'UV is bad for your skin.'
         self.poutput(result)
 
-    def _onchange_degrees_c(self, param_name, old, new) -> None:
+    def _onchange_degrees_c(self, _param_name, _old, new) -> None:
         # if it's over 40C, it's gotta be sunny, right?
         if new > 40:
             self.sunny = True

examples/migrating.py

@@ -13,7 +13,7 @@ class CmdLineApp(cmd.Cmd):
     MUMBLE_FIRST = ('so', 'like', 'well')
     MUMBLE_LAST = ('right?',)
 
-    def do_exit(self, line) -> bool:
+    def do_exit(self, _line) -> bool:
         """Exit the application."""
         return True
 

examples/modular_commands/commandset_basic.py

@@ -73,7 +73,7 @@ def do_raise_error(self, statement: Statement) -> None:
         """Demonstrates effect of raising CompletionError."""
         self._cmd.poutput(f"Args: {statement.args}")
 
-    def complete_raise_error(self, text: str, line: str, begidx: int, endidx: int) -> list[str]:
+    def complete_raise_error(self, _text: str, _line: str, _begidx: int, _endidx: int) -> list[str]:
         """CompletionErrors can be raised if an error occurs while tab completing.
 
         Example use cases
@@ -83,5 +83,5 @@ def complete_raise_error(self, text: str, line: str, begidx: int, endidx: int) -
         raise CompletionError("This is how a CompletionError behaves")
 
     @with_category('Not Basic Completion')
-    def do_custom_category(self, statement: Statement) -> None:
+    def do_custom_category(self, _statement: Statement) -> None:
         self._cmd.poutput('Demonstrates a command that bypasses the default category')

examples/modular_commands/commandset_complex.py

@@ -7,10 +7,10 @@
 
 @cmd2.with_default_category('Fruits')
 class CommandSetA(cmd2.CommandSet):
-    def do_apple(self, statement: cmd2.Statement) -> None:
+    def do_apple(self, _statement: cmd2.Statement) -> None:
         self._cmd.poutput('Apple!')
 
-    def do_banana(self, statement: cmd2.Statement) -> None:
+    def do_banana(self, _statement: cmd2.Statement) -> None:
         """Banana Command."""
         self._cmd.poutput('Banana!!')