ci(check-ci-skip): fix commitMessagesMetadata.forEach is not a function

Primary Changes
----------------
1. Added condition to commitMessagesMetadata
to check if commit message is array or not.

Fixes #3614

Signed-off-by: bado <[email protected]>

tools/ci-skip-for-maintainers.js

@@ -15,13 +15,13 @@ const MAINTAINERS_REGEX = new RegExp(
 const main = async () => {
   const markdownContent = readFileSync(MaintainersFile, "utf-8");
 
-  const args = process.argv.slice(2);
-  const pullReqUrl = args[0];
-  const committerLogin = args[1];
+  //const args = process.argv.slice(2);
+  //const pullReqUrl = args[0];
+  //const committerLogin = args[1];
 
   //Uncomment these lines and change it for local machine testing purposes:
-  //const pullReqUrl = "https://api.github.com/repos/<username>/cactus/pulls/<number>";
-  //const committerLogin = "<username>";
+  const pullReqUrl = "https://api.github.com/repos/zondervancalvez/cactus/pulls/7";
+  const committerLogin = "zondervancalvez";
 
   const fetchJsonFromUrl = async (url) => {
     const fetchResponse = await fetch(url);
@@ -33,10 +33,14 @@ const main = async () => {
     pullReqUrl + "/commits",
   );
 
-  commitMessagesMetadata.forEach((commitMessageMetadata) => {
-    // get commit message body
-    commitMessageList.push(commitMessageMetadata["commit"]["message"]);
-  });
+  if (Array.isArray(commitMessagesMetadata)) {
+    commitMessagesMetadata.forEach((commitMessageMetadata) => {
+      // get commit message body
+      commitMessageList.push(commitMessageMetadata["commit"]["message"]);
+    });
+  } else {
+    console.error("Commit message data is empty or null");
+  }
 
   // Check if skip-ci is found in commit message
   const checkSkipCI = () => {