Update code-review.yml
This commit is contained in:
29
.github/workflows/code-review.yml
vendored
29
.github/workflows/code-review.yml
vendored
@@ -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 형식으로 제공 부탁드립니다
|
||||
6. **중첩 최소화** (이중 이상 분기·반복문 지양)
|
||||
7. **중괄호 생략 금지**
|
||||
8. **하드 코딩 금지**
|
||||
9. **미사용 코드 제거**
|
||||
|
||||
위반사항이 없으면 `None`이라고 답변해 주세요.
|
||||
Reference in New Issue
Block a user