style: minor comment changes

Author: tomli380576

Tools/PC/c3-submission-helpers/parse-suspend-30-logs.py

@@ -105,7 +105,9 @@ class Meta(TypedDict):
 
 
 def parse_args() -> Input:
-    p = argparse.ArgumentParser()
+    p = argparse.ArgumentParser(
+        formatter_class=argparse.ArgumentDefaultsHelpFormatter
+    )
     p.add_argument(
         "-s",
         "--no-summary",
@@ -124,7 +126,7 @@ def parse_args() -> Input:
     p.add_argument(
         "filenames",
         nargs="+",
-        help="The path to the suspend logs or the stress test submission .tar",
+        help="The path to the stress test submission .tar files",
     )
     p.add_argument(
         "-w",
@@ -139,13 +141,13 @@ def parse_args() -> Input:
         "-d",
         "--directory",
         dest="write_dir",
-        default="",
+        default=os.getcwd(),
         required=False,
         help=(
-            "Where to write the individual logs. "
-            "If not specified and the -w flag is true, "
-            "it will create a new local directory called "
-            "{your original file name}-split."
+            "The top level dir of where to write the individual logs. "
+            "Inside this directory, subdirectories called "
+            "{your original file name}-split will be created to contain the "
+            "individual .txt files of each run"
         ),
     )
     p.add_argument(
@@ -158,7 +160,7 @@ def parse_args() -> Input:
     p.add_argument(
         "-nb",
         "--num-boots",
-        help="Set the expected number of boots in the input file. Default=3.",
+        help="Set the expected number of boots in the input file.",
         dest="num_boots",
         default=3,
         required=False,
@@ -167,7 +169,7 @@ def parse_args() -> Input:
     p.add_argument(
         "-ns",
         "--num-suspends-per-boot",
-        help="Set the expected number of runs in the input file. Default=30.",
+        help="Set the expected number of runs in the input file.",
         dest="num_suspends",
         default=30,
         required=False,

Tools/PC/c3-submission-helpers/summarize-reboot-check-test.py

@@ -77,9 +77,6 @@ def bold(self, s: str):
         return f"\033[1m{s}\033[0m"
 
 
-C = Color()
-
-
 class Log:
     @staticmethod
     def ok(*args: str):
@@ -672,7 +669,8 @@ def _group_by_index(self):
 def parse_args() -> Input:
     p = argparse.ArgumentParser(
         description="Parses the outputs of reboot_check_test.py "
-        "from a C3 submission tar file"
+        "from a C3 submission tar file",
+        formatter_class=argparse.ArgumentDefaultsHelpFormatter,
     )
     p.add_argument(
         "filenames",
@@ -711,9 +709,9 @@ def parse_args() -> Input:
         dest="expected_n_runs",
         help=(
             "Specify a value to show a warning when the number of boot files "
-            "!= the number of runs you expect. Default=30. "
+            "!= the number of runs you expect. "
             "Note that this number applies to both cold and warm boot "
-            "since checkbox doesn't use a different number for CB/WB either."
+            "since checkbox doesn't use a different number for CB/WB. "
         ),
         type=int,
         default=30,
@@ -729,7 +727,8 @@ def parse_args() -> Input:
 def main():
     args = parse_args()
 
-    C.no_color = args.no_color
+    global C
+    C = Color(no_color=args.no_color)
 
     for filename in args.filenames:
         reader = SubmissionTarReader(filename)