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

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

Merged
merged 3 commits into from
Jan 14, 2024
Merged

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

merged 3 commits into from
Jan 14, 2024

Conversation

imseongtae
Copy link
Owner

No description provided.

@imseongtae imseongtae self-assigned this Jan 14, 2024
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_ptemperature: 이러한 매개 변수의 값을 조정하여 결과의 다양성과 신뢰성 간의 균형을 조정하세요.

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

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

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_ptemperature: 기본값으로 설정된 값일 경우 작동하지만, 다른 값들을 테스트하여 최적의 사용자 경험을 찾아볼 수 있습니다.

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

@imseongtae imseongtae merged commit 8d468cf into main Jan 14, 2024
1 check passed
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.

1 participant