Added some return type hints

Author: kmvanbrunt

cmd2/cmd2.py

@@ -1024,7 +1024,7 @@ def path_complete(self, text: str, line: str, begidx: int, endidx: int,
         """
 
         # Used to complete ~ and ~user strings
-        def complete_users():
+        def complete_users() -> List[str]:
 
             # We are returning ~user strings that resolve to directories,
             # so don't append a space or quote in the case of a single result.
@@ -2198,7 +2198,7 @@ def _cmdloop(self) -> bool:
 
     # -----  Alias sub-command functions -----
 
-    def alias_create(self, args: argparse.Namespace):
+    def alias_create(self, args: argparse.Namespace) -> None:
         """Create or overwrite an alias"""
 
         # Validate the alias name
@@ -2223,7 +2223,7 @@ def alias_create(self, args: argparse.Namespace):
         self.aliases[args.name] = value
         self.poutput("Alias '{}' {}".format(args.name, result))
 
-    def alias_delete(self, args: argparse.Namespace):
+    def alias_delete(self, args: argparse.Namespace) -> None:
         """Delete aliases"""
         if args.all:
             self.aliases.clear()
@@ -2238,7 +2238,7 @@ def alias_delete(self, args: argparse.Namespace):
                 else:
                     self.perror("Alias '{}' does not exist".format(cur_name), traceback_war=False)
 
-    def alias_list(self, args: argparse.Namespace):
+    def alias_list(self, args: argparse.Namespace) -> None:
         """List some or all aliases"""
         if args.name:
             for cur_name in utils.remove_duplicates(args.name):
@@ -2314,7 +2314,7 @@ def alias_list(self, args: argparse.Namespace):
 
     # Preserve quotes since we are passing strings to other commands
     @with_argparser(alias_parser, preserve_quotes=True)
-    def do_alias(self, args: argparse.Namespace):
+    def do_alias(self, args: argparse.Namespace) -> None:
         """Manage aliases"""
         func = getattr(args, 'func', None)
         if func is not None:
@@ -2326,7 +2326,7 @@ def do_alias(self, args: argparse.Namespace):
 
     # -----  Macro sub-command functions -----
 
-    def macro_create(self, args: argparse.Namespace):
+    def macro_create(self, args: argparse.Namespace) -> None:
         """Create or overwrite a macro"""
 
         # Validate the macro name
@@ -2401,7 +2401,7 @@ def macro_create(self, args: argparse.Namespace):
         self.macros[args.name] = Macro(name=args.name, value=value, minimum_arg_count=max_arg_num, arg_list=arg_list)
         self.poutput("Macro '{}' {}".format(args.name, result))
 
-    def macro_delete(self, args: argparse.Namespace):
+    def macro_delete(self, args: argparse.Namespace) -> None:
         """Delete macros"""
         if args.all:
             self.macros.clear()
@@ -2416,7 +2416,7 @@ def macro_delete(self, args: argparse.Namespace):
                 else:
                     self.perror("Macro '{}' does not exist".format(cur_name), traceback_war=False)
 
-    def macro_list(self, args: argparse.Namespace):
+    def macro_list(self, args: argparse.Namespace) -> None:
         """List some or all macros"""
         if args.name:
             for cur_name in utils.remove_duplicates(args.name):
@@ -2517,7 +2517,7 @@ def macro_list(self, args: argparse.Namespace):
 
     # Preserve quotes since we are passing strings to other commands
     @with_argparser(macro_parser, preserve_quotes=True)
-    def do_macro(self, args: argparse.Namespace):
+    def do_macro(self, args: argparse.Namespace) -> None:
         """Manage macros"""
         func = getattr(args, 'func', None)
         if func is not None: