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

Commit b952c00

Browse files
Updating workflow to send reminder and removing time dependency.
1 parent 3b8e3f4 commit b952c00

File tree

1 file changed

+9
-62
lines changed

1 file changed

+9
-62
lines changed

.github/workflows/heretto-reminder.yml

Lines changed: 9 additions & 62 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ name: Pull Request Reminder
22

33
on:
44
pull_request:
5-
types: [opened, synchronize, reopened]
5+
types: [opened, synchronize]
66

77
jobs:
88
reminder:
@@ -15,67 +15,14 @@ jobs:
1515
checks: write
1616
pull-requests: write
1717
steps:
18-
- name: Checkout code
19-
uses: actions/checkout@v4
20-
21-
- name: Get Pull Request Info
22-
id: pr_info
23-
uses: actions/github-script@v7
24-
with:
25-
script: |
26-
const { owner, repo, number } = context.issue;
27-
const pr = await github.rest.pulls.get({
28-
owner,
29-
repo,
30-
pull_number: number,
31-
});
32-
33-
const author = pr.data.user.login;
34-
const labels = pr.data.labels.map(label => label.name);
35-
console.log("Labels from PR Info:", labels); // Added debug log
36-
return { author, number, labels };
37-
38-
- name: Check if Reminder Needed
39-
id: check_reminder
40-
uses: actions/github-script@v7
18+
- name: Add PR Comment
19+
uses: actions/github-script@v6
4120
with:
21+
github-token: ${{ secrets.MY_TOKEN }}
4222
script: |
43-
const { labels } = ${{ steps.pr_info.outputs }};
44-
const reminderLabel = 'reminder sent';
45-
let sendReminder = false;
46-
if (!Array.isArray(labels) || labels.length === 0) {
47-
return { sendReminder: true };
48-
} else if (labels.includes(reminderLabel)) {
49-
return { sendReminder: false };
50-
} else {
51-
return { sendReminder: true };
52-
}
53-
console.log("sendReminder value:", sendReminder); // Added debug log
54-
return { sendReminder: sendReminder };
55-
56-
- name: Send Reminder Comment
57-
if: ${{ (steps.check_reminder.outputs.sendReminder == true) && (toJSON(steps.check_reminder.outputs)) }}
58-
uses: actions/github-script@v7
59-
with:
60-
script: |
61-
const { owner, repo, number, author } = ${{ steps.pr_info.outputs }};
62-
const message = `@${author}, please make sure that you update the files in Heretto and post the Heretto share link in this PR.`;
6323
await github.rest.issues.createComment({
64-
owner,
65-
repo,
66-
issue_number: number,
67-
body: message,
68-
});
69-
70-
- name: Add Reminder Label
71-
if: ${{ (steps.check_reminder.outputs.sendReminder == true) && (toJSON(steps.check_reminder.outputs)) }}
72-
uses: actions/github-script@v7
73-
with:
74-
script: |
75-
const { owner, repo, number } = ${{ steps.pr_info.outputs }};
76-
await github.rest.issues.addLabels({
77-
owner,
78-
repo,
79-
issue_number: number,
80-
labels: ['reminder sent'],
81-
});
24+
owner: context.repo.owner,
25+
repo: context.repo.repo,
26+
issue_number: context.payload.pull_request.number,
27+
body: "`@${owner}, please make sure that you update the files in Heretto and post the Heretto share link in this PR.`"
28+
})

0 commit comments

Comments
 (0)