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

Detect console.logs and its equivalent in other languages #324

Merged
merged 15 commits into from
Oct 18, 2023

Conversation

baristaGeek
Copy link
Collaborator

Description

The goal of this PR is to detect console.log()s and its equivalent in other languages (System.out.println(), println(), print(), fmt.Println(), etc.). The initial goal is to successfully detect such. Then we can comment them and add them to the PR Rating score.

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

Notes

I am initially sending this PR with a simple console.log() to test this PR.

Acceptance

@vercel
Copy link

vercel bot commented Oct 11, 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 Oct 18, 2023 7:50pm

@watermelon-copilot-for-code-review
Copy link

watermelon-copilot-for-code-review bot commented Oct 11, 2023

WatermelonAI Summary

The 15 commits in this Pull Request are focused on the business logic of detecting and handling console.log() statements in code. Commits 1-5 remove or comment out console.log() statements, commit 6-9 update prompts and remove old logic, commit 10 restricts the rating to a maximum of 8 if console log comments are found, commit 11 updates a utility function, commit 12 introduces a new octokit call, commit 14 concatenates with different syntax, and commit 15 fixes a merge conflict. Overall, these commits aim to enhance the detection and handling of console.log() statements in the codebase.

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.
🍉🫶

Copy link
Member

@EstebanDalelR EstebanDalelR left a comment

Choose a reason for hiding this comment

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

This is a test, right?
Does not seem to work.

@baristaGeek
Copy link
Collaborator Author

This is a test, right? Does not seem to work.

As stated in the description, yes

@watermelon-copilot-for-code-review

This PR contains console logs. Please remove them.

@watermelon-copilot-for-code-review

This PR contains console logs in file app/api/actions/github/route.ts. Please remove them.

@watermelon-copilot-for-code-review

This PR contains console logs in file utils/actions/labelPullRequest.ts. Please remove them.

Co-authored-by: Esteban Dalel R <estebandalelr@gmail.com>
@watermelon-copilot-for-code-review

This PR contains console logs in file utils/actions/detectConsoleLogs.ts. Please remove them.

@watermelon-copilot-for-code-review

This PR contains console logs in file utils/actions/labelPullRequest.ts. Please remove them.

@watermelon-copilot-for-code-review

This PR contains console logs in file app/api/actions/github/route.ts. Please remove them.

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