SC-83560: Fix getting commit hash (#1)

neofinancial · Feb 8, 2022 · 0ffd2ee · 0ffd2ee
1 parent 82a30dc
commit 0ffd2ee
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 9 deletions.
diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS
@@ -0,0 +1 @@
+* @iansu
diff --git a/src/get-data.ts b/src/get-data.ts
@@ -25,20 +25,20 @@ const getData = async (): Promise<PRData> => {
   const info = context.payload;
 
   if (info.pull_request) {
-    prData.ref = `${info.pull_request.head.ref}`;
-    prData.baseRef = `${info.pull_request.base.ref}`;
-    prData.sha = `${info.pull_request.merge_commit_sha}`;
-    prData.timestamp = `${info.pull_request.created_at}`;
+    prData.ref = info.pull_request.head.ref;
+    prData.baseRef = info.pull_request.base.ref;
+    prData.sha = info.pull_request.head.sha;
+    prData.timestamp = info.pull_request.created_at;
     prData.pullRequest = info.number;
   } else {
-    prData.ref = `${info.ref.replace('refs/heads/', '')}`;
-    prData.baseRef = `${info.base_ref}`;
-    prData.sha = `${info.after}`;
-    prData.timestamp = `${info.head_commit.timestamp}`;
+    prData.ref = info.ref.replace('refs/heads/', '');
+    prData.baseRef = info.base_ref;
+    prData.sha = info.after;
+    prData.timestamp = info.head_commit.timestamp;
   }
 
   if (info.repository && info.sender) {
-    prData.actor = `${info.sender.login}`;
+    prData.actor = info.sender.login;
   } else {
     throw new Error('Issue with context');
   }