Skip to content
This repository has been archived by the owner on Mar 12, 2024. It is now read-only.

Bugfix: Remove duplicate PR comments #362

Merged
merged 4 commits into from
Nov 7, 2023
Merged

Bugfix: Remove duplicate PR comments #362

Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
822ed7a
Add random files
baristaGeek Nov 3, 2023
c5e9d46
Add a space to WatermelonAI summary title
baristaGeek Nov 3, 2023
af00550
Search for new app name to avoid duplicate comments
baristaGeek Nov 3, 2023
a340702
Remove test files (I messed up)
baristaGeek Nov 3, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions app/api/actions/github/route.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -337,7 +337,7 @@ export async function POST(request: Request) {

const count = await addActionCount({ owner });

textToWrite += `### WatermelonAI Summary \n`;
textToWrite += `### Watermelon AI Summary \n`;
let businessLogicSummary;
if (AISummary) {
businessLogicSummary = await getOpenAISummary({
Expand Down Expand Up @@ -469,7 +469,7 @@ export async function POST(request: Request) {
);
// Find our bot's comment
let botComment = comments.data.find((comment) => {
return comment?.user?.login.includes("watermelon-context");
return comment?.user?.login.includes("watermelon-copilot-for-code-review");
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did we change the name of the bot?
When?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When we agreed to change the messaging of the product and the company as a whole

});
if (botComment?.id) {
// Update the existing comment
Expand Down