Enable ruff PIE ruleset and fix related issues

Author: tleonhardt

cmd2/ansi.py

@@ -212,12 +212,10 @@ class FgColor(AnsiSequence):
     """Base class for ANSI Sequences which set foreground text color"""
 
 
-
 class BgColor(AnsiSequence):
     """Base class for ANSI Sequences which set background text color"""
 
 
-
 ####################################################################################
 # Implementations intended for direct use
 ####################################################################################

cmd2/exceptions.py

@@ -16,7 +16,6 @@ class SkipPostcommandHooks(Exception):
     """
 
 
-
 class Cmd2ArgparseError(SkipPostcommandHooks):
     """
     A ``SkipPostcommandHooks`` exception for when a command fails to parse its arguments.
@@ -26,15 +25,13 @@ class Cmd2ArgparseError(SkipPostcommandHooks):
     """
 
 
-
 class CommandSetRegistrationError(Exception):
     """
     Exception that can be thrown when an error occurs while a CommandSet is being added or removed
     from a cmd2 application.
     """
 
 
-
 class CompletionError(Exception):
     """
     Raised during tab completion operations to report any sort of error you want printed. This can also be used
@@ -84,17 +81,13 @@ class Cmd2ShlexError(Exception):
     """Raised when shlex fails to parse a command line string in StatementParser"""
 
 
-
 class EmbeddedConsoleExit(SystemExit):
     """Custom exception class for use with the py command."""
 
 
-
 class EmptyStatement(Exception):
     """Custom exception class for handling behavior when the user just presses <Enter>."""
 
 
-
 class RedirectionError(Exception):
     """Custom exception class for when redirecting or piping output fails"""
-

examples/default_categories.py

@@ -15,7 +15,6 @@ class MyBaseCommandSet(CommandSet):
     """Defines a default category for all sub-class CommandSets"""
 
 
-
 class ChildInheritsParentCategories(MyBaseCommandSet):
     """
     This subclass doesn't declare any categories so all commands here are also categorized under 'Default Category'

plugins/ext_test/tasks.py

@@ -144,7 +144,6 @@ def dist_clean(context):
 def clean_all(context):
     """Run all clean tasks"""
     # pylint: disable=unused-argument
-    pass
 
 
 namespace_clean.add_task(clean_all, 'all')

tests/test_argparse.py

@@ -293,9 +293,9 @@ def base_helpless(self, args):
     # This subcommand has aliases and no help text. It exists to prevent changes to _set_parser_prog() which
     # use an approach which relies on action._choices_actions list. See comment in that function for more
     # details.
-    parser_bar = base_subparsers.add_parser('helpless', aliases=['helpless_1', 'helpless_2'])
-    parser_bar.add_argument('z', help='string')
-    parser_bar.set_defaults(func=base_bar)
+    parser_helpless = base_subparsers.add_parser('helpless', aliases=['helpless_1', 'helpless_2'])
+    parser_helpless.add_argument('z', help='string')
+    parser_helpless.set_defaults(func=base_bar)
 
     @cmd2.with_argparser(base_parser)
     def do_base(self, args):

tests/test_argparse_completer.py

@@ -1263,10 +1263,10 @@ def _subcmd_no_custom(self, args: argparse.Namespace) -> None:
         pass
 
     # Parser for a subcommand with a custom completer type
-    custom_completer_parser = Cmd2ArgumentParser(description="Custom completer", ap_completer_type=CustomCompleter)
-    custom_completer_parser.add_argument('--myflag', complete_when_ready=True)
+    custom_completer_parser2 = Cmd2ArgumentParser(description="Custom completer", ap_completer_type=CustomCompleter)
+    custom_completer_parser2.add_argument('--myflag', complete_when_ready=True)
 
-    @cmd2.as_subcommand_to('top', 'custom', custom_completer_parser, help="custom completer")
+    @cmd2.as_subcommand_to('top', 'custom', custom_completer_parser2, help="custom completer")
     def _subcmd_custom(self, args: argparse.Namespace) -> None:
         pass
 

tests_isolated/test_commandset/test_argparse_subcommands.py

@@ -50,9 +50,9 @@ def base_helpless(self, args):
     # This subcommand has aliases and no help text. It exists to prevent changes to _set_parser_prog() which
     # use an approach which relies on action._choices_actions list. See comment in that function for more
     # details.
-    parser_bar = base_subparsers.add_parser('helpless', aliases=['helpless_1', 'helpless_2'])
-    parser_bar.add_argument('z', help='string')
-    parser_bar.set_defaults(func=base_bar)
+    parser_helpless = base_subparsers.add_parser('helpless', aliases=['helpless_1', 'helpless_2'])
+    parser_helpless.add_argument('z', help='string')
+    parser_helpless.set_defaults(func=base_bar)
 
     @cmd2.with_argparser(base_parser)
     def do_base(self, args):