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

Conversation

baristaGeek
Copy link
Collaborator

@baristaGeek baristaGeek commented Nov 3, 2023

Description
We want to improve security and salt access tokens.

Type of change
Bug fix (non-breaking change which fixes an issue)

Acceptance
I have read the Contributing guidelines
I have read the Code of Conduct

Copy link

vercel bot commented Nov 3, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
watermelon ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 3, 2023 5:33pm

test.md Outdated
@@ -0,0 +1 @@
console.log("hello test");

Choose a reason for hiding this comment

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

This file contains at least one console log. Please remove any present.

test.md Outdated
@@ -0,0 +1 @@
console.log("hello test");

Choose a reason for hiding this comment

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

This file contains at least one console log. Please remove any present.

test.md Outdated
@@ -0,0 +1 @@
console.log("hello test");

Choose a reason for hiding this comment

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

This file contains at least one console log. Please remove any present.

test.md Outdated
@@ -0,0 +1 @@
console.log("hello test");

Choose a reason for hiding this comment

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

This file contains at least one console log. Please remove any present.

Watermelon AI Summary

This PR adds random files to the repository. This suggests that the business logic involves the need for additional files, potentially for testing purposes or to support new functionality.

GitHub PRs

Click here to login to Jira
Click here to login to Confluence
Click here to login to Slack
No results found in Notion Pages :(

No results found in Linear Tickets :(

Click here to login to Asana
watermelon is an open repo and Watermelon will serve it for free.
🍉🫶

test.md Outdated
@@ -0,0 +1 @@
console.log("hello test");

Choose a reason for hiding this comment

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

This file contains at least one console log. Please remove any present.

WatermelonAI Summary

This PR includes three commits: adding random files, adding a space to the WatermelonAI summary title, and searching for a new app name. It aims to enhance the app by avoiding duplicate comments and improving overall functionality.

GitHub PRs

Click here to login to Jira
Click here to login to Confluence
Click here to login to Slack
No results found in Notion Pages :(

No results found in Linear Tickets :(

Click here to login to Asana
watermelon is an open repo and Watermelon will serve it for free.
🍉🫶

WatermelonAI Summary

This PR includes several commits that update the code by adding random files, adding a space to the WatermelonAI summary title, searching for a new app name to avoid duplicate comments, and removing test files. These changes improve the codebase and align with the business logic of maintaining a clean and organized code repository.

GitHub PRs

Click here to login to Jira
Click here to login to Confluence
Click here to login to Slack
No results found in Notion Pages :(

No results found in Linear Tickets :(

Click here to login to Asana
watermelon is an open repo and Watermelon will serve it for free.
🍉🫶

@baristaGeek baristaGeek changed the title [TEST] - Add random files Bugfix: Remove duplicate PR comments Nov 3, 2023
@@ -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

@baristaGeek baristaGeek merged commit 0fc9057 into dev Nov 7, 2023
3 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants