Await tracking

Author: zegl

.eslintrc.json

@@ -47,7 +47,8 @@
       "@typescript-eslint/type-annotation-spacing": "error",
       "@typescript-eslint/unbound-method": "error",
       "no-console": "off",
-      "import/no-commonjs": "off"
+      "import/no-commonjs": "off",
+      "github/no-then": "off"
     },
     "env": {
       "node": true,

src/approver/main.ts

@@ -36,7 +36,7 @@ async function run(): Promise<void> {
 }
 
 run()
-  .then(() => track({jobID, actionName: 'approver'}))
+  .then(async () => await track({jobID, actionName: 'approver'}))
   .catch(error => {
     if (error instanceof Error) {
       track({jobID, actionName: 'approver', error: error.message})

src/baller/main.ts

@@ -22,12 +22,12 @@ async function run(): Promise<{jobId: string}> {
 }
 
 run()
-  .then(({jobId}) => {
-    track({jobID: jobId, actionName: 'baller'})
+  .then(async ({jobId}) => {
+    await track({jobID: jobId, actionName: 'baller'})
     core.setOutput('codeball-job-id', jobId)
   })
-  .catch(error => {
-    track({actionName: 'baller', error: error.message})
+  .catch(async error => {
+    await track({actionName: 'baller', error: error.message})
     if (error instanceof Error) {
       core.setFailed(error.message)
     }

src/labeler/main.ts

@@ -73,10 +73,10 @@ const run = async (): Promise<void> => {
 }
 
 run()
-  .then(() => track({jobID, actionName: 'labeler'}))
-  .catch(error => {
+  .then(async () => await track({jobID, actionName: 'labeler'}))
+  .catch(async error => {
     if (error instanceof Error) {
-      track({jobID, actionName: 'labeler', error: error.message})
+      await track({jobID, actionName: 'labeler', error: error.message})
       if (error.message === 'Resource not accessible by integration') {
         core.setFailed(
           'Codeball Labeler failed to access GitHub. Check the "GITHUB_TOKEN Permissions" of this job and make sure that the job has WRITE permissions to Pull Requests.'

src/status/main.ts

@@ -28,13 +28,13 @@ const run = async (jobID: string): Promise<boolean> => {
 const jobID = required('codeball-job-id')
 
 run(jobID)
-  .then(isApproved => {
-    track({jobID, actionName: 'status'})
-    core.setOutput('approved', isApproved)
+  .then(async approved => {
+    await track({jobID, actionName: 'status'})
+    core.setOutput('approved', approved)
   })
-  .catch(error => {
+  .catch(async error => {
     if (error instanceof Error) {
-      track({jobID, actionName: 'status', error: error.message})
+      await track({jobID, actionName: 'status', error: error.message})
       core.setFailed(error.message)
     }
   })