Add config for CodeRabbit review#4890
Conversation
Signed-off-by: Tomoyuki Morita <moritato@amazon.com>
Signed-off-by: Tomoyuki Morita <moritato@amazon.com>
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing touches🧪 Generate unit tests (beta)
Comment |
| enabled: false # Disabled auto-review until it becomes stable | ||
| auto_incremental_review: false | ||
| drafts: false # Don't review draft PRs | ||
| ignore_title_keywords: |
There was a problem hiding this comment.
Nice, I think we can also start exercising this in our PR template when we initially creating PR.
| 1. **Code Clarity**: Is the code self-explanatory? | ||
| 2. **Test Coverage**: Are there adequate tests? | ||
| 3. **Error Handling**: Are errors handled appropriately? | ||
| 4. **Documentation**: Is JavaDoc complete and accurate? |
There was a problem hiding this comment.
Is this rule specific for command development?
There was a problem hiding this comment.
No, this is generic for any code change in this repository.
| - Test compatibility with Calcite version constraints | ||
|
|
||
| chat: | ||
| auto_reply: true |
There was a problem hiding this comment.
Not sure if I like this setting, maybe others like it though?
Usually when responding to AI comments the only audience is other reviewers (e.g. "this comment doesn't apply" or "implemented"), I'm not sure I see the value of the AI reply
* Add review instructions for CodeRabbit Signed-off-by: Tomoyuki Morita <moritato@amazon.com> * Disable auto review Signed-off-by: Tomoyuki Morita <moritato@amazon.com> --------- Signed-off-by: Tomoyuki Morita <moritato@amazon.com>
Description
opensearch-project/sql.github#412)Related Issues
#4889
opensearch-project/.github#412
Check List
--signoffor-s.By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.