Fix #280: Fix bug with multi-cased user names (#281)

oppia · Jul 27, 2021 · 1c45852 · 1c45852
1 parent eea2542
commit 1c45852
Show file tree

Hide file tree

Showing 3 changed files with 130,613 additions and 132,735 deletions.
diff --git a/actions/src/pull_requests/claCheck.js b/actions/src/pull_requests/claCheck.js
@@ -112,7 +112,10 @@ const checkSheet = async (auth) => {
         core.setFailed('No data found.');
       } else {
         core.info(`Checking if ${PR_AUTHOR} has signed the CLA`);
-        const hasUserSignedCla = flatRows.includes(PR_AUTHOR);
+        const hasUserSignedCla = flatRows.some(
+          username => username.toLowerCase() === PR_AUTHOR.toLowerCase()
+        );
+
         await generateOutput(hasUserSignedCla);
       }
     }