Print argparse's help msg if invalid input

Author: bjk7119

src/fosslight_oss_pkg/_convert.py

@@ -52,17 +52,21 @@ def convert_report(base_path, output_name, format, need_log_file=True):
             files_to_convert = base_path.split(",")
             file_option_on = True
             for file in files_to_convert:
-                if file.endswith((".yaml", ".yml")):
-                    oss_yaml_files.append(file)
+                if os.path.isfile(file):
+                    if file.endswith((".yaml", ".yml")):
+                        oss_yaml_files.append(file)
+                    else:
+                        logger.error("Not support file name or extension")
+                        sys.exit(1)
                 else:
-                    logger.error("Not support file name or extension")
+                    logger.error(f"Check the path to find: {base_path}")
                     sys.exit(1)
 
     if oss_yaml_files:
         convert_yml_to_excel(oss_yaml_files, output_report, file_option_on, base_path)
     else:
         logger.info("fosslight_prechecker: can't convert anything")
-        logger.info("Try 'fosslight_prechecker -h for more information")
+        logger.info("Try 'fosslight_prechecker -h' for more information")
 
     try:
         _str_final_result_log = safe_dump(_result_log, allow_unicode=True, sort_keys=True)

src/fosslight_prechecker/_help.py

@@ -4,6 +4,7 @@
 # SPDX-License-Identifier: GPL-3.0-only
 from fosslight_util.help import PrintHelpMsg
 
+
 _HELP_MESSAGE_PRECHECKER = """
     FOSSLight Prechecker is a tool that checks whether source code complies with copyright and license writing rules.
 

src/fosslight_prechecker/_precheck.py

@@ -281,5 +281,5 @@ def run_lint(target_path, disable, output_file_name, format='', need_log_file=Tr
             logger.warning("Can't make result file\n")
         sys.exit(exit_code)
     else:
-        logger.error(f"Check the path to find : {path_to_find}")
+        logger.error(f"Check the path to find : {target_path}")
         sys.exit(1)

src/fosslight_prechecker/cli.py

@@ -14,22 +14,22 @@
 
 def main():
     parser = argparse.ArgumentParser(description='FOSSLight Prechecker', prog='fosslight_prechecker', add_help=False)
-    parser.add_argument('mode', nargs='?', help='lint | convert | add', choices=['lint', 'convert', 'add'])
-    parser.add_argument('--path', '-p', help='Path to check', type=str, dest='path', default="")
-    parser.add_argument('--format', '-f', help='Format of ouput', type=str, dest='format', default="")
-    parser.add_argument('--output', '-o', help='Output file name', type=str, dest='output', default="")
-    parser.add_argument('--no', '-n', help='Disable automatic exclude mode', action='store_true', dest='disable')
-    parser.add_argument('--license', '-l', help='License name to add', type=str, dest='license', default="")
-    parser.add_argument('--copyright', '-c', help='Copyright to add', type=str, dest='copyright', default="")
-    parser.add_argument('--help', '-h', help='Print help message', action='store_true', dest='help')
-    parser.add_argument('--ignore', '-i', help='Do not write log to file', action='store_false', dest='log')
-    parser.add_argument('--version', '-v', help='Print FOSSLight Prechecker version', action='store_true', dest='version')
+    parser.add_argument('mode', nargs='?', help='lint | convert | add', choices=['lint', 'add', 'convert'])
+    parser.add_argument('-h', '--help', help='Print help message', action='store_true', dest='help')
+    parser.add_argument('-i', '--ignore', help='Do not write log to file', action='store_false', dest='log')
+    parser.add_argument('-v', '--version', help='Print FOSSLight Prechecker version', action='store_true', dest='version')
+    parser.add_argument('-n', '--no', help='Disable automatic exclude mode', action='store_true', dest='disable')
+    parser.add_argument('-p', '--path', help='Path to check', type=str, dest='path', default="")
+    parser.add_argument('-f', '--format', help='Format of ouput', type=str, dest='format', default="")
+    parser.add_argument('-o', '--output', help='Output file name', type=str, dest='output', default="")
+    parser.add_argument('-l', '--license', help='License name to add', type=str, dest='license', default="")
+    parser.add_argument('-c', '--copyright', help='Copyright to add', type=str, dest='copyright', default="")
     try:
-        args, unknown = parser.parse_known_args()
+        args = parser.parse_args()
     except SystemExit:
-        print_help_msg()
+        sys.exit(0)
 
-    if args.help or unknown:
+    if args.help:
         print_help_msg()
 
     if args.version: