refactor: case sensitivity and require brackets handling in validate.ts

tsinis · Apr 9, 2024 · bb8cdb5 · bb8cdb5
1 parent e1bf18e
commit bb8cdb5
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 15 deletions.
diff --git a/dist/index.js b/dist/index.js
diff --git a/src/validate.ts b/src/validate.ts
@@ -48,12 +48,16 @@ export async function validate({
   let strategy: Strategy | undefined = undefined;
   fallbackToAsk = fallbackToAsk !== undefined ? fallbackToAsk : false;
   const client = github.getOctokit(token, octokitOpts);
+  caseSensitive = String(caseSensitive).toLowerCase() === "true";
+  console.log(`Case Sensitive: ${caseSensitive}`);
+  requireBrackets = String(requireBrackets).toLowerCase() === "true";
+  console.log(`Require Brackets: ${requireBrackets}`);
   const regex = buildRegexPattern(
     shipKeyword || Strategy.Ship,
     showKeyword || Strategy.Show,
     askKeyword || Strategy.Ask,
-    requireBrackets !== undefined ? requireBrackets : true,
-    caseSensitive !== undefined ? caseSensitive : false,
+    requireBrackets,
+    caseSensitive,
   );
 
   try {
@@ -66,17 +70,10 @@ export async function validate({
     });
 
     console.log(`Regex: ${regex}`);
-    console.log(`Case Sensitive: ${caseSensitive}`);
     const title = pr.title.trim().normalize().replace(/"/g, "");
     console.log(`Actual PR Title Before Match: "${title}"`);
     const match = title.match(regex);
     console.log(`Match Result: ${match}`);
-    console.log(
-      title
-        .split("")
-        .map((c) => c.charCodeAt(0).toString(16))
-        .join(" "),
-    );
 
     if (match) {
       // Extract the keyword from the match