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

make a rough draft of identifying if the bot already commented #364

Closed
wants to merge 1 commit into from
Closed

make a rough draft of identifying if the bot already commented #364

wants to merge 1 commit into from

Conversation

baristaGeek
Copy link
Collaborator

Description

WIP. It's a little bit more challenging to prevent dup comments on the line diff than it's to do it on the PR.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Chore: cleanup/renaming, etc
  • RFC

Acceptance

@vercel Vercel
Copy link

vercel bot commented Nov 3, 2023 •
edited
Loading

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 6:08pm

watermelon-copilot-for-code-review[bot]
watermelon-copilot-for-code-review bot reviewed Nov 3, 2023
View reviewed changes
utils/actions/detectConsoleLogs.ts
comment.user.login === "watermelon-copilot-for-code-review";
});

if (addtionsHaveConsoleLog === "true" && existingCommentsByBot.length === 0) {
const commentFileDiff = () => {
Copy link

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-copilot-for-code-review Watermelon - Copilot For Code Review
Copy link

WatermelonAI Summary

This PR aims to create a draft for identifying if the bot has already commented. It states that preventing duplicate comments on a line diff is more challenging than doing it on the PR itself. The PR body also includes acceptance of contributing guidelines and code of conduct.

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 :(

Linear Tickets

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

This was referenced Nov 6, 2023
Merged
Merged
Merged
Closed
@watermelon-copilot-for-code-review watermelon-copilot-for-code-review bot mentioned this pull request Nov 14, 2023
Merged
8 tasks
@baristaGeek
Copy link
Collaborator Author

#377 is the correct approach

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@watermelon-copilot-for-code-review watermelon-copilot-for-code-review[bot] watermelon-copilot-for-code-review[bot] left review comments

Assignees
No one assigned
Labels
🍉 Safe to Merge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@baristaGeek