Skip to content
This repository was archived by the owner on Sep 2, 2025. It is now read-only.

Commit c520985

Browse files
Updating workflow to send reminder and removing time dependency.
1 parent fc10c97 commit c520985

File tree

1 file changed

+3
-13
lines changed

1 file changed

+3
-13
lines changed

.github/workflows/heretto-reminder.yml

Lines changed: 3 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -31,35 +31,25 @@ jobs:
3131
});
3232
3333
const author = pr.data.user.login;
34-
const createdAt = new Date(pr.data.created_at);
35-
const now = new Date();
36-
const diffInDays = Math.floor((now - createdAt) / (1000 * 60 * 60 * 24));
3734
const labels = pr.data.labels.map(label => label.name);
3835
39-
return { author, diffInDays, number, labels };
36+
return { author, number, labels };
4037
4138
- name: Check if Reminder Needed
4239
id: check_reminder
4340
uses: actions/github-script@v7
4441
with:
4542
script: |
46-
const { diffInDays, labels } = ${{ steps.pr_info.outputs }};
43+
const { labels } = ${{ steps.pr_info.outputs }};
4744
const reminderLabel = 'reminder sent';
48-
const daysThreshold = 1; // Number of days before sending reminder. Adjust as needed.
4945
5046
if (!labels) { // Check if labels is undefined or null
51-
return { sendReminder: diffInDays >= daysThreshold }; // send reminder if the time threshold is passed, even without labels.
47+
return { sendReminder: true }; // send reminder if the time threshold is passed, even without labels.
5248
}
5349
if (labels.includes(reminderLabel)) {
5450
return { sendReminder: false };
5551
}
5652
57-
if (diffInDays >= daysThreshold) {
58-
return { sendReminder: true };
59-
} else {
60-
return { sendReminder: false };
61-
}
62-
6353
- name: Send Reminder Comment
6454
if: ${{ steps.check_reminder.outputs.sendReminder == 'true' }}
6555
uses: actions/github-script@v7

0 commit comments

Comments
 (0)