modify approach to handling help

HopkinsIDD · Dec 11, 2024 · e6ced61 · e6ced61
1 parent b5142d9
commit e6ced61
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 3 deletions.
diff --git a/flepimop/gempyor_pkg/src/gempyor/cli.py b/flepimop/gempyor_pkg/src/gempyor/cli.py
@@ -21,7 +21,7 @@
 
 
 # add some basic commands to the CLI
-@cli.command(params=[config_files_argument] + list(config_file_options.values()))
+@cli.command(params=[config_files_argument] + list(config_file_options.values()), context_settings=dict(help_option_names = ['-h', '--help']))
 @click.pass_context
 def patch(ctx: click.Context = mock_context, **kwargs) -> None:
     """Merge configuration files

diff --git a/flepimop/gempyor_pkg/src/gempyor/shared_cli.py b/flepimop/gempyor_pkg/src/gempyor/shared_cli.py
@@ -222,7 +222,9 @@ def _parse_option(param: click.Parameter, value: Any) -> Any:
     config_src = []
     if len(found_configs) != 1:
         if not found_configs:
-            raise ValueError(f"No config files provided.")
+            click.echo("No configuration provided! See help for required usage:\n")
+            click.echo(ctx.get_help())
+            ctx.exit()
         else:
             error_dict = {k: kwargs[k] for k in found_configs}
             raise ValueError(

diff --git a/flepimop/gempyor_pkg/src/gempyor/simulate.py b/flepimop/gempyor_pkg/src/gempyor/simulate.py
@@ -300,7 +300,8 @@ def simulate(
 
 
 @cli.command(
-    name="simulate", params=[config_files_argument] + list(config_file_options.values())
+    name="simulate", params=[config_files_argument] + list(config_file_options.values()),
+    context_settings=dict(help_option_names=["-h", "--help"])
 )
 @pass_context
 def _click_simulate(ctx: Context, **kwargs) -> int: