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

ci: add code review bot #1250

Merged
merged 1 commit into from
Mar 24, 2023
Merged

ci: add code review bot #1250

merged 1 commit into from
Mar 24, 2023

Conversation

ysfscream
Copy link
Member

PR Checklist

If you have any questions, you can refer to the Contributing Guide

What is the current behavior?

Please describe the current behavior and link to a relevant issue.

Issue Number

Example: #123

What is the new behavior?

Please describe the new behavior or provide screenshots.

Does this PR introduce a breaking change?

  • Yes
  • No

Specific Instructions

Are there any specific instructions or things that should be known prior to review?

Other information

@ysfscream ysfscream added the chore Changes in build tools or dependent packages label Mar 24, 2023
@ysfscream ysfscream self-assigned this Mar 24, 2023
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
OPENAI_API_KEY: ${{ secrets.OPENAI_API_KEY }}
LANGUAGE: English

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:

  1. Permission: The code looks fine, it uses the necessary permission for reading and writing pull requests.

  2. On Section: It is good that it uses the pull_request trigger with the types opened and reopened.

  3. Jobs Section: It looks good that it uses ubuntu-latest and uses the ChatGPT-CodeReview@v1 action.

  4. Environment Variables: It looks good that it uses GITHUB_TOKEN and OPENAI_API_KEY as secrets and LANGUAGE as English.

Overall, the code looks good and meets the necessary requirements.

@ysfscream ysfscream merged commit 22e5bdd into main Mar 24, 2023
@ysfscream ysfscream deleted the dev/packages branch March 24, 2023 03:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Changes in build tools or dependent packages
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants