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.

Sign up for GitHub

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

Jump to bottom

👷 (chore): Code review 액션 추가 #18

Merged
merged 3 commits into from
Jan 14, 2024
Merged

👷 (chore): Code review 액션 추가 #18

Changes from 2 commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
a10410c
👷 (chore): Code review 액션 추가
imseongtae Jan 14, 2024
9578d7b
👷 (chore): max_tokens 항목 삭제
imseongtae Jan 14, 2024
5fea68a
👷 (chore): GITHUB_TOKEN 을 사용하도록 수정
imseongtae Jan 14, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
26 changes: 26 additions & 0 deletions .github/workflows/cr.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,26 @@
name: Code Review

permissions:
contents: read
pull-requests: write

on:
pull_request:
types: [opened, reopened, synchronize]

jobs:
test:
# if: ${{ contains(github.event.*.labels.*.name, 'gpt review') }} # Optional; to run only when a label is attached
runs-on: ubuntu-latest
steps:
- uses: anc95/ChatGPT-CodeReview@main
env:
GITHUB_TOKEN: ${{ secrets.GH_TOKEN }}
OPENAI_API_KEY: ${{ secrets.OPENAI_API_KEY }}
# Optional
LANGUAGE: Korean
OPENAI_API_ENDPOINT: https://api.openai.com/v1
MODEL: gpt-3.5-turbo # https://platform.openai.com/docs/models
PROMPT: # example: Please check if there are any confusions or irregularities in the following code diff:
top_p: 1 # https://platform.openai.com/docs/api-reference/chat/create#chat/create-top_p
temperature: 1 # https://platform.openai.com/docs/api-reference/chat/create#chat/create-temperature
Copy link
Owner Author

Choose a reason for hiding this comment

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

이 코드 패치는 GitHub에서 Code Review 작업을 자동화하기 위한 템플릿으로 보입니다. 몇 가지 개선 및 버그 위험 요소가 있습니다:

  1. if 문 주석 처리: #를 제거하여 코드 블록 실행 조건을 설정합니다.
  2. LANGUAGE 환경 변수: 사용 가능한 언어 옵션을 확인하고 적절한 값으로 설정하세요.
  3. PROMPT 환경 변수: 적절한 프롬프트 메시지를 제공하세요.
  4. top_p 및 temperature: 이러한 매개 변수의 값을 조정하여 결과의 다양성과 신뢰성 간의 균형을 조정하세요.

더욱 개선할 부분이 있는지 자세히 분석하기 위해서는 anc95/ChatGPT-CodeReview@main 액션의 소스 코드를 살펴봐야 합니다.

Copy link

Choose a reason for hiding this comment

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

이 코드 패치에 대해 간략한 코드 리뷰를 도와드리겠습니다. 버그 위험과 개선 제안을 환영합니다.

  1. LANGUAGE 변수: 번역된 주석에서 한국어로 응답을 원하므로 필요하지 않은 옵션입니다. 삭제하는 것이 좋습니다.
  2. MODEL 변수: gpt-3.5-turbo는 상당히 뛰어난 모델이지만 비용이 높은 선택입니다. 작업의 목적에 따라 더 가벼운 모델로 변경할 수 있는지 확인하세요.
  3. PROMPT 변수: 주석 예시가 필요한 경우 이 변수를 사용할 수 있지만, 현재는 비어 있으므로 삭제해도 좋습니다.
  4. top_p와 temperature: 기본값으로 설정된 값일 경우 작동하지만, 다른 값들을 테스트하여 최적의 사용자 경험을 찾아볼 수 있습니다.

이외에는 보안이나 성능과 관련된 문제는 없어보입니다. 이러한 수정 사항을 고려하여 코드를 개선할 수 있습니다.