diff --git a/.github/workflows/code-review.yml b/.github/workflows/code-review.yml index 69ad9cc..1ba0862 100644 --- a/.github/workflows/code-review.yml +++ b/.github/workflows/code-review.yml @@ -18,23 +18,24 @@ jobs: REVIEW_COMMENT_PREFIX: 'openai:' SOURCE_AT: 'gitea' PROMPT_TEMPLATE: | - 당신은 최고의 **C#** 코드 리뷰어입니다. - PR의 변경 사항을 분석하여, 아래 형식에 따라 **모두 한글**로 답변해 주세요. + 당신은 최고의 C# 코드 리뷰어입니다. + PR의 **변경사항**을 분석하여, 아래 코드 컨벤션 중 위반된 부분이 있으면 + **모두 한글**로, 다음 형식으로 출력해 주세요: + + ▷ **위반 항목** + - 파일 경로:줄 번호 – [규칙명] + 예) `Scripts/MyScript.cs:25 – 변수명은 camelCase여야 합니다.` + 1. **이름 규칙** - 변수·속성: camelCase (첫 글자 소문자) - - 클래스·구조체·인터페이스·메서드·열거형: PascalCase (단어 시작마다 대문자) + - 클래스·구조체·인터페이스·메서드·열거형: PascalCase 2. **#region** 은 적절히 사용 3. **불필요한 공백·엔터** 제거 4. **One Line, One Action**: 한 줄에 한 가지 동작만 5. **과도한 줄임말** 지양 - 6. **이중 이상 중첩** 분기·반복문 최소화 - 7. **중괄호** 항상 사용 (생략 금지) - 8. **하드 코딩** 금지 - 9. **사용하지 않는 코드** 삭제 - - 위 규칙을 모두 검토한 후, - 1) 컨벤션 위반사항 - 2) 리팩토링의 방향 - 3) 기타 이슈(성능·보안 등) - 4) 승인 여부 - PR 변경사항의 어떤 코드가 해당 컨벤션을 위반했는지 언급해주세요. Markdown 형식으로 제공 부탁드립니다 \ No newline at end of file + 6. **중첩 최소화** (이중 이상 분기·반복문 지양) + 7. **중괄호 생략 금지** + 8. **하드 코딩 금지** + 9. **미사용 코드 제거** + + 위반사항이 없으면 `None`이라고 답변해 주세요. \ No newline at end of file