Merge pull request #4 from sprucehealth/jon/set-output

set outputs (didSetStatus and statusFieldValue)
sprucehealth · Sep 13, 2024 · af82229 · af82229
2 parents 93424d5 + f0b9332
commit af82229
Show file tree

Hide file tree

Showing 4 changed files with 33 additions and 39 deletions.
diff --git a/action.yml b/action.yml
@@ -42,6 +42,11 @@ inputs:
     required: false
     description: 'If specified, the PR will be labeled with this value when the status is updated to the value of `statusFieldValueWhenPRReadyForReviewIsApproved`.'
     default: ''
+outputs:
+  didSetStatus:
+    description: 'true if the status was set, false or undefined if not set'
+  statusFieldValue:
+    description: 'the value of the status field that was set, or undefined if not set'
 runs:
   using: 'node20'
   main: 'dist/index.js'
diff --git a/dist/index.js b/dist/index.js
diff --git a/dist/main.js b/dist/main.js
diff --git a/src/main.ts b/src/main.ts
@@ -122,6 +122,8 @@ export async function run(): Promise<void> {
         continue;
       }
 
+      let fieldValue = '';
+
       if (
         // this is expected to run upon PRs being opened or reopened
         triggerIsPullRequest
@@ -189,12 +191,7 @@ export async function run(): Promise<void> {
             skipSettingStatusForPRReadyForReviewIsApprovedIfLabeledWith.length > 0 &&
             !hasSkipSettingStatusForPRApprovedLabel
           ) {
-            await setStatusFieldvalueForAsanaTask({
-              fieldValue: statusFieldValueWhenPRReadyForReviewIsApproved,
-              taskID,
-              client,
-              statusCustomField,
-            });
+            fieldValue = statusFieldValueWhenPRReadyForReviewIsApproved;
           }
           if (labelToApplyToPRWhenApproved) {
             await octokit.issues.addLabels({
@@ -205,32 +202,30 @@ export async function run(): Promise<void> {
             });
           }
         } else if (pr?.draft && statusFieldValueWhenDraftPRIsOpen) {
-          await setStatusFieldvalueForAsanaTask({
-            fieldValue: statusFieldValueWhenDraftPRIsOpen,
-            taskID,
-            client,
-            statusCustomField,
-          });
+          fieldValue = statusFieldValueWhenDraftPRIsOpen;
         } else if (statusFieldValueWhenPRReadyForReviewIsOpen) {
-          await setStatusFieldvalueForAsanaTask({
-            fieldValue: statusFieldValueWhenPRReadyForReviewIsOpen,
-            taskID,
-            client,
-            statusCustomField,
-          });
+          fieldValue = statusFieldValueWhenPRReadyForReviewIsOpen;
         }
       } else if (
         // this is expected to run on pushes to `main` (aka a merged pull request)
         triggerIsPushToMain &&
         statusFieldValueForMergedCommitToMain
       ) {
         core.info(`🔍 triggerIsPushToMain`);
+        fieldValue = statusFieldValueForMergedCommitToMain;
+      }
+
+      if (fieldValue) {
         await setStatusFieldvalueForAsanaTask({
-          fieldValue: statusFieldValueForMergedCommitToMain,
+          fieldValue,
           taskID,
           client,
           statusCustomField,
         });
+        core.setOutput('didSetStatus', 'true');
+        core.setOutput('statusFieldValue', fieldValue);
+      } else {
+        core.setOutput('didSetStatus', 'false');
       }
     }
   } catch (error) {