Updating workflow to send reminder and removing time dependency.

jcatera-splunk · jcatera-splunk · commit d0b3cfff530b · 2025-03-20T17:01:18.000-07:00
diff --git a/.github/workflows/heretto-reminder.yml b/.github/workflows/heretto-reminder.yml
@@ -43,7 +43,7 @@ jobs:
             const { labels } = ${{ steps.pr_info.outputs }};
             const reminderLabel = 'reminder sent';
 
-            if (!labels) { // Check if labels is undefined or null
+            if (!labels || labels.length === 0) { 
               return { sendReminder: true }; // send reminder if the time threshold is passed, even without labels.
             }
             if (labels.includes(reminderLabel)) {

Original file line number	Diff line number	Diff line change
`@@ -43,7 +43,7 @@ jobs:`
`43`	`43`	`const { labels } = ${{ steps.pr_info.outputs }};`
`44`	`44`	`const reminderLabel = 'reminder sent';`
`45`	`45`
`46`		`- if (!labels) { // Check if labels is undefined or null`
	`46`	`+ if (!labels \|\| labels.length === 0) {`
`47`	`47`	`return { sendReminder: true }; // send reminder if the time threshold is passed, even without labels.`
`48`	`48`	`}`
`49`	`49`	`if (labels.includes(reminderLabel)) {`