Explicitly define default case for "store_true" argparse

Argparse "store_true" action implicitly defaults to false. Explicitly define false as the default case and update the help message. Signed-off-by: Anderson Chauphan <[email protected]>
trilinos · Dec 4, 2024 · a6ee402 · a6ee402
1 parent 8999222
commit a6ee402
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/packages/framework/pr_tools/PullRequestLinuxDriverTest.py b/packages/framework/pr_tools/PullRequestLinuxDriverTest.py
@@ -199,9 +199,10 @@ def parse_args():
     optional.add_argument('--skip-create-packageenables',
                           dest="skip_create_packageenables",
                           action="store_true",
+                          default=False,
                           help="Skip the creation of the packageEnables.cmake fragment file generated by " + \
                           "the TriBITS infrastructure indicating which packages are to be enabled based on file " + \
-                          "changes between a source and target branch.  Default=")
+                          "changes between a source and target branch. Default=False")
 
     desc_subprojects_file = "The subprojects_file is used by the testing infrastructure. This parameter " + \
                             "allows the default, generated file, to be overridden. Generally this should " + \