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

infra: 코드리뷰 봇 설정 #10

Merged
merged 6 commits into from
Jun 14, 2024
Merged

Conversation

chansooo
Copy link
Member

@chansooo chansooo commented Jun 14, 2024

What is this PR? 🔍

이슈

resolves #11

설명

  • 코드리뷰 봇 추가

Changes 📝

  • 코드리뷰 봇 추가

Screenshot 📸

image

To Reviewers 🙏

귀엽죠?

temperature: 1
max_tokens: 4096
MAX_PATCH_LENGTH: 10000

Choose a reason for hiding this comment

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

코드 리뷰 다음과 같은 관점으로 검토하였습니다:

버그 위험성:

  1. env 설정: GITHUB_TOKENOPENAI_API_KEY와 같이 중요한 환경 변수가 제대로 설정되지 않으면 스크립트가 정상 작동하지 않을 수 있습니다. 환경 변수가 제대로 설정되었는지 확인하세요.

개선 사항 제안:

  1. 주석 정리: PROMPT에 대한 주석이 미사용 상태로 남아있습니다. 불필요한 주석은 제거하거나 필요한 경우 활성화하는 것이 좋습니다.

        # PROMPT: # example: Please check if there are any confusions or irregularities in the following code diff:

    위 부분을 삭제하거나, 구현할 필요가 있다면 적절히 바꾸세요.

  2. 변수명 일관성: MAX_PATCH_LENGTH 관련 변수가 대문자로 되어 있는데, 다른 변수들 (top_p, temperature, max_tokens)은 소문자로 되어 있어 일관성이 부족합니다. 모두 소문자 또는 대문자로 통일하는 것이 가독성 측면에서 좋습니다.

          max_patch_length: 10000
  3. 의미 있는 주석 추가: 각 단계와 의도에 대해 더 자세히 기술된 주석을 추가하면 코드 유지보수성이 높아집니다.

  4. 출력 정보 추가: 작업의 성공 여부를 확인하기 위해 로그 출력을 추가하는 것도 고려할 만 합니다.

대체적으로 코드가 잘 구성되어 있으나, 상기사항을 반영한다면 더욱 향상될 것이라 생각합니다.

@chansooo chansooo merged commit 1478758 into develop Jun 14, 2024
4 checks passed
@chansooo chansooo deleted the infra/코드리뷰-봇-설정 branch June 14, 2024 11:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature] 코드리뷰 봇 적용
1 participant