run-drc-for-cell-gds-using-magic: Rename match argument.

Rename `match-directories` argument to `match-cells-directories` to
better describe what it actually does.

Signed-off-by: Tim 'mithro' Ansell <tansell@google.com>

run-drc-for-cell-gds-using-magic/action.yml

diff --git a/run-drc-for-cell-gds-using-magic/action.yml b/run-drc-for-cell-gds-using-magic/action.yml
index 1a842d2..d1ddf17 100644
--- a/run-drc-for-cell-gds-using-magic/action.yml
+++ b/run-drc-for-cell-gds-using-magic/action.yml
@@ -28,9 +28,9 @@
     description: >-
         A file containing a list of newline-delimited acceptable DRC errors.
     default: /dev/null
-  match-directories:
+  match-cell-directories:
     description: >-
-        A regex that will match subdirectories under cells to be checked.
+        A regex that will match cell names to be checked.
     default: ^.*$
   known-bad:
     description: >-
@@ -46,7 +46,7 @@
   - ${{ inputs.top }}
   - --acceptable-errors-file
   - ${{ inputs.acceptable-errors-file }}
-  - --match-directories
-  - ${{ inputs.match-directories }}
+  - --match-cell-directories
+  - ${{ inputs.match-cell-directories }}
   - --known-bad
   - ${{ join(inputs.known-bad, ',') }}

run-drc-for-cell-gds-using-magic/run_all_drc.py

diff --git a/run-drc-for-cell-gds-using-magic/run_all_drc.py b/run-drc-for-cell-gds-using-magic/run_all_drc.py
index 8af9bc2..a091cb7 100644
--- a/run-drc-for-cell-gds-using-magic/run_all_drc.py
+++ b/run-drc-for-cell-gds-using-magic/run_all_drc.py
@@ -150,10 +150,11 @@
 )
 @click.option(
     "-m",
-    "--match-directories",
+    "--match-cell-directories",
     default="^.*$",
-    help="A regex that will match subdirectories under cells/."
-         " Default: . (matches everything.)"
+    help="A regex that that will match cell names to be checked (which will"
+         " match subdirectories under cells/)."
+         " Default: ^.*$ (matches everything)"
 )
 @click.option(
     "-b",
@@ -163,10 +164,16 @@
          " thus do not cause a non-zero exit upon failure."
          " Default: empty string (None of them.)"
 )
-def run_all_drc(top, acceptable_errors_file, match_directories, known_bad):
+def run_all_drc(
+            top,
+            acceptable_errors_file,
+            match_cell_directories,
+            known_bad,
+        ):
+
     os.chdir(top)
     print("Testing cells in %s directories matching /%s/…" % (
-        os.getcwd(), match_directories))
+        os.getcwd(), match_cell_directories))
 
     global acceptable_errors
     acceptable_errors_str = open(acceptable_errors_file).read()
@@ -182,7 +189,7 @@
         cells = os.listdir(cells_dir)
 
         for cell in cells:
-            if not re.fullmatch(match_directories, cell):
+            if not re.fullmatch(match_cell_directories, cell):
                 print("Skipping directory %s…" % cell)
                 continue
 
@@ -204,24 +211,23 @@
             total += 1
             cell_name, errors = future.result()
 
-            if cell_name in known_bad_list:
-                symbol = "✘\ufe0f"
-                message = "ERROR (ignored as known bad)"
-            else:
-                symbol = "❌"
-                message = "ERROR"
-
             if len(errors) == 0:
                 successes += 1
                 # This tick is rendered black on all major platforms except for
                 # Microsoft.
                 symbol = "✔\ufe0f"
                 message = "CLEAN"
+            elif cell_name in known_bad_list:
+                symbol = "✘\ufe0f"
+                message = "ERROR (ignored as known bad)"
+            else:
+                symbol = "❌"
+                message = "ERROR"
+                exit_code = 65
+
             print("%-64s %s %s" % (cell_name, symbol, message))
 
             if len(errors) != 0:
-                if cell_name not in known_bad_list:
-                    exit_code = 65
                 for error in errors:
                     print("* %s" % error[0])
                     for line in error[1]: