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

PR comments support #495

Open
Tradunsky opened this issue Nov 7, 2023 · 3 comments
Open

PR comments support #495

Tradunsky opened this issue Nov 7, 2023 · 3 comments

Comments

@Tradunsky
Copy link

I like the idea of AI Agent supported code review! Giving a try to code rabbit ai now.

Generally want to reduce time when code review is triggered and focus it only on times, when humans also asked for code review.

One of the use cases highlighted is sometimes users create a PR and iteratively keep adding to it until it is fully ready for code review and even after code review changing code as well.

In that use case, the preference is to have AI code review just once. No need for summaries, release notes, nothing except code review and non trivial suggestions.

Right now using PR level comment trigger the action is skipped with the message:

Warning: Skipped: this action only works on push events or pull_request

Would it be possible add the support for PR level comments that would simply scan all changes in PR?

Here is the full trigger of the workflow we have:

on:
  issue_comment:
    types: [created]
  pull_request_review_comment:
    types: [created]
@guritfaq
Copy link
Collaborator

CodeRabbit Pro supports on-demand reviews but this doesn’t exist in OSS yet.
In Pro, automatic reviews can be disabled and reviews can be triggered manually using review commands.

@Tradunsky
Copy link
Author

Thank you for the explanation @guritfaq 🙏🏻

The review commands in code place do work as expected. However, when I run a review via PR level comment the Github Action application prevents this from being triggered. Even though the Pro version might indeed be more comprehensive, do you think it is still possible to let the open source variant to be triggered when it is best for community users?

@guritfaq
Copy link
Collaborator

@Tradunsky We are a small team and have to prioritize requests for our Pro version at this stage. However, we will add this to our backlog and address it as soon as possible.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants