Rename do_reflect option to do-reflect

We should use dashes instead of underscores in command line options. This confirms with the [Gnu style guide](https://www.gnu.org/software/libc/manual/html_node/Argument-Syntax.html)
Aggregate-Intellect · Nov 16, 2023 · 1386ce9 · 1386ce9
1 parent b34e838
commit 1386ce9
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/sherpa_ai/config/task_config.py b/src/sherpa_ai/config/task_config.py
@@ -41,7 +41,7 @@ def from_config(cls, configs: List[str]) -> "AgentConfig":
             help="site to be used for the Google search tool.",
         )
         parser.add_argument(
-            "--do_reflect",
+            "--do-reflect",
             action="store_true",
             help="enable performing the reflection step for each agent.",
         )

diff --git a/src/tests/unit_tests/config/test_task_config.py b/src/tests/unit_tests/config/test_task_config.py
@@ -5,7 +5,7 @@
 
 def test_all_parameters_parse_successfully():
     site = "https://www.google.com"
-    input_str = f"Test input. --verbose --gsite {site} --do_reflect"
+    input_str = f"Test input. --verbose --gsite {site} --do-reflect"
 
     parsed, config = AgentConfig.from_input(input_str)