Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GitHub Action: Code Rabbit Approval Check #3003

Closed
palisadoes opened this issue Dec 28, 2024 · 1 comment
Closed

GitHub Action: Code Rabbit Approval Check #3003

palisadoes opened this issue Dec 28, 2024 · 1 comment
Assignees
Labels
dependencies Pull requests that update a dependency file feature request good first issue Good for newcomers security Security fix test Testing application

Comments

@palisadoes
Copy link
Contributor

Is your feature request related to a problem? Please describe.

  1. We have a policy of not merging PRs unless CodeRabbit.ai approves the PR
  2. This requires a manual verification which takes time and is unreliable
  3. We need a GitHub Action workflow that automates this

With this check we will only need to review PRs that have green ticks that are truly valid.

Describe the solution you'd like

  1. CodeRabbit.ai usually approves the PR before the workflow run is complete
  2. We need to have a check at the end of the PR workflow that checks whether the PR has been approved
  3. If it is not approved, then the check must fail. Ideally it should check the name of the reviewer as being the CodeRabbit user
  4. The test for starting the app in docker and without docker must always pass first. This is a mandatory dependency

Describe alternatives you've considered

  • N/A

Approach to be followed (optional)

  • N/A

Additional context

  • N/A

Potential internship candidates

Please read this if you are planning to apply for a Palisadoes Foundation internship

@github-actions github-actions bot added dependencies Pull requests that update a dependency file test Testing application unapproved good first issue Good for newcomers security Security fix labels Dec 28, 2024
@mandeepnh5
Copy link
Contributor

Can i work on this issue?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file feature request good first issue Good for newcomers security Security fix test Testing application
Projects
Archived in project
Development

No branches or pull requests

2 participants