Feature/comment specific line diffs #331
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
WatermelonAI SummaryThe 13 commits in this Pull Request indicate that the team is working on commenting on specific diff lines that contain console logs. Initially, they added hello world examples in Python, C++, and Rust for testing purposes. They then made updates and fixes to the code and implemented the functionality to detect console logs and comment on them using the Octokit route. They intentionally sent a PR with console logs to test the functionality, but later removed the console logs and test hello worlds. There were also some minor changes made, such as fixing a typo and changing the event from REQUEST_CHANGES to COMMENT. Overall, this PR introduces a new feature that allows for commenting on specific lines in the code diff that contain console logs. GitHub PRsClick here to login to Jira |
watermelon-copilot-for-code-review
bot
added
🚨 Don't Merge
👀 Take a deeper dive
and removed
🚨 Don't Merge
👀 Take a deeper dive
labels
watermelon-copilot-for-code-review
bot
added
🚨 Don't Merge
and removed
👀 Take a deeper dive
labels
watermelon-copilot-for-code-review
bot
added
👀 Take a deeper dive
🚨 Don't Merge
and removed
🚨 Don't Merge
👀 Take a deeper dive
labels
![watermelon-copilot-for-code-review[bot]](https://avatars.githubusercontent.com/in/327523?s=60&v=4){kind=small}
watermelon-copilot-for-code-review
bot
added
🍉 Safe to Merge
and removed
👀 Take a deeper dive
labels
utils/actions/detectConsoleLogs.ts
Outdated
| const indiviudalLine = openAIResult.line; | ||
|
|
||
| console.log("additionsHaveConsoleLog", addtionsHaveConsoleLog); | ||
| console.log("indiviudalLine", indiviudalLine); |
There was a problem hiding this comment.
getConsoleLogPosition - This file contains at least one console log. Please remove any present.
watermelon-copilot-for-code-review
bot
added
👀 Take a deeper dive
and removed
🍉 Safe to Merge
labels
…for testing purposes
test_hello_worlds/hello.rs
Outdated
| @@ -0,0 +1,3 @@ | |||
| fn main() { | |||
| println!("Hello World!"); | |||
| } | |||
There was a problem hiding this comment.
getConsoleLogPosition - This file contains at least one console log. Please remove any present.
watermelon-copilot-for-code-review
bot
added
🚨 Don't Merge
and removed
👀 Take a deeper dive
labels
baristaGeek
changed the title
Description
This PR uses the console log detection function to comment on the specific diff in the specific file containing the console log.
To do so, we tell OpenAI to also return the code line itself, and then get the code line in the position of the diff (not the actual file) which is what Octokit needs.
Type of change
Acceptance