Closed pranshugupta54 closed 1 week ago
The recent updates introduce a new .coderabbit.yaml
configuration file for integrating CodeRabbit AI, modify GitHub workflow checks to include this new file, and update environment variable names in INSTALLATION.md
to use underscores instead of camel case.
Files | Change Summary |
---|---|
.coderabbit.yaml | Added configuration settings for language server integration, reviews, and chat settings. |
.github/workflows/pull-request.yml | Included .prettierrc , .prettierignore , and .coderabbit.yaml in the list of files checked for unauthorized changes. |
INSTALLATION.md | Updated environment variable names from camel case to underscores in the .env file setup instructions. |
Objective (Issue #1990) | Addressed | Explanation |
---|---|---|
Configure using CodeRabbit YAML File | ✅ | |
Using the docs - https://docs.coderabbit.ai/guides/configure-coderabbit/ | ✅ |
In a burrow deep, where code does sleep,
A rabbit hops with secrets to keep.
YAML files and workflows tight,
Variables set just right.
With every change, it leaps and bounds,
Through fields of code, where harmony resounds.
🐇✨
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
We have these basic policies to make the approval process smoother for our volunteer team.
Please make sure your code passes all tests. Our test code coverage system will fail if these conditions occur:
The process helps maintain the overall reliability of the code base and is a prerequisite for getting your PR approved. Assigned reviewers regularly review the PR queue and tend to focus on PRs that are passing.
Do not assign reviewers. Our Queue Monitors will review your PR and assign them. When your PR has been assigned reviewers contact them to get your code reviewed and approved via:
Your reviewer(s) will have the following roles:
Read our CONTRIBUTING.md file. Most importantly:
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 98.26%. Comparing base (
7091c76
) to head (f81abab
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@pranshugupta54
develop
branch and include references to main
main
. We can do this as we are not yet live.Can this run in develop using the YAML file? We need the workflow to be triggered with PRs to that branch, not just main
.
With some of the other GitHub actions you need add a secret to do this.
@palisadoes, I'll make same for develop for syncing.
base_branches:
- develop
This part is for auto review on develop branch (Including the default main
- so we don't need to mention it separately)
What kind of change does this PR introduce?
Feature
Issue Number:
Fixes #1990
Did you add tests for your changes?
NA
Snapshots/Videos:
NA
If relevant, did you update the documentation?
NA
Summary
Added YAML Config file for CodeRabbit AI using docs- https://docs.coderabbit.ai/guides/configure-coderabbit
Does this PR introduce a breaking change?
NA
Other information
NA
Have you read the contributing guide?
Yes
Summary by CodeRabbit
New Features
Documentation
INSTALLATION.md
to reflect new environment variable naming conventions.Chores
.prettierrc
,.prettierignore
, and.coderabbit.yaml
in workflow checks for unauthorized changes.