Moves DangerfileRunner to use async methods.

danger · Oct 1, 2017 · 2f59cf3 · 2f59cf3
1 parent 4b498f0
commit 2f59cf3
Showing 1 changed file with 18 additions and 2 deletions.
diff --git a/source/runner/DangerfileRunner.ts b/source/runner/DangerfileRunner.ts
@@ -94,9 +94,25 @@ export async function runDangerfileEnvironment(
 ): Promise<DangerResults> {
   const vm = new NodeVM(environment)
 
+  const fetchContents = async () => {
+    return new Promise<string>((resolve, reject) => {
+      if (originalContents) {
+        resolve(originalContents)
+      } else {
+        fs.readFile(filename, "utf8", (error, data) => {
+          if (error) {
+            reject(error)
+          } else {
+            resolve(data)
+          }
+        })
+      }
+    })
+  }
+
   // Require our dangerfile
-  originalContents = originalContents || fs.readFileSync(filename, "utf8")
-  let content = cleanDangerfile(originalContents)
+  const fetchedContents = await fetchContents()
+  let content = cleanDangerfile(fetchedContents)
 
   // TODO: Relative imports get TS/Babel