search

coderabbitai / ai-pr-reviewer

AI-based Pull Request Summarizer and Reviewer with Chat Capabilities.
https://coderabbit.ai
MIT License
1.46k stars 277 forks source link

Add more context lines to each patch #332

Closed pistachiomatt closed 1 year ago

pistachiomatt commented 1 year ago

Often, the PR reviewer will make false assumptions because it can only see a small amount of lines that have been changed in the file.

One common issue is it'll say "This variable isn't being used anywhere" when it in fact is, it's just referenced outside the hunk.

It would be useful to add a config to allow larger hunks/more context lines.

guritfaq commented 1 year ago

Hi @pistachiomatt , Thanks for the feedback. We are launching a professional version of this open-source project, where we will be focusing our future development. We invite you to check out Coderabbit.ai to learn more.

We have noted this issue and plan on addressing this in the coming weeks.

pistachiomatt commented 1 year ago

That's great. Congrats!

pistachiomatt commented 1 year ago

Update this ticket when you get to it. I'd be keen to switch over to your paid plan then (but not before because it's not very useful until that feature is built.)

guritfaq commented 1 year ago

Thank you. Will keep you posted.

harjotgill commented 1 year ago

Noise should be minimal outside the patch lines since we pushed prompt changes a few weeks back. In addition, the Pro version of CodeRabbit has better noise reduction, and we haven't seen these kinds of noisy comments lately. Closing ticket.