Separate headSha from triggerSha (#790)

iterative · Oct 29, 2021 · 569e548 · 569e548 · github-actions · Oct 29, 2021
1 parent 3037be7
commit 569e548
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/src/cml.js b/src/cml.js
@@ -83,8 +83,12 @@ class CML {
   }
 
   async headSha() {
+    return exec(`git rev-parse HEAD`);
+  }
+
+  async triggerSha() {
     const { sha } = getDriver(this);
-    return sha || (await exec(`git rev-parse HEAD`));
+    return sha || (await this.headSha());
   }
 
   async branch() {
@@ -165,7 +169,7 @@ class CML {
   }
 
   async checkCreate(opts = {}) {
-    const { headSha = await this.headSha() } = opts;
+    const { headSha = await this.triggerSha() } = opts;
 
     return await getDriver(this).checkCreate({ ...opts, headSha });
   }
@@ -334,7 +338,7 @@ class CML {
       return;
     }
 
-    const sha = await this.headSha();
+    const sha = await this.triggerSha();
     const shaShort = sha.substr(0, 8);
 
     const target = await this.branch();