-
Notifications
You must be signed in to change notification settings - Fork 2
Git Convention
jeongjun Hwang edited this page Oct 12, 2022
·
3 revisions
- 각자 코드를 올리는 브랜치 이름은 자신의 이름초성을 딴 알파벳으로 한다.
- ex) 황정준 ->
HJJ
- 한주동안 사용했던 브랜치는 PR을 날린 후 merge되면 삭제해주고(local 브랜치도 삭제해야함) 다시 같은 이름의 브랜치를 생성해 진행
-
커밋 메시지는 한글로 간결하게 작성한다. (자유롭게)
-
ex) 유기농 배추 문제풀이
- PR 제목
-
문제풀이 PR :
[WeekNN] 홍길동 문제풀이 업로드(ex -[Week32] 황정준 문제풀이 업로드) -
그 외 PR : 자유롭게
- PR 라벨
-
Solve- 모든 문제를 풀었을 경우 -
Not Solve- 한문제라도 못 푼 문제가 있을 경우 -
Java- Java로 문제를 푼 경우 -
Python- Python으로 문제를 푼 경우
ex)

우측에 보이는 Lavel 옆 톱니바퀴를 클릭해 적절한 라벨을 추가해준 뒤
Create Pull Request버튼을 눌러준다.
- PR merge는 자신을 제외한 4명의 코드리뷰 후 승인이 있어야 merge 할 수 있음
-
코드리뷰를 원하는 PR에 들어가서
File changed클릭
-
올라온 코드를 홛인해서 좋은 코드나 개선할 수 있을 것 같은 코드엔 코맨트를 남겨줌 (참고 - 코드리뷰하는법)
-
코드를 다 확인했으면
File changed탭의 우측 상단에Review changes클릭후Approve체크후 리뷰 최종 제출
