팀원 각자의 코드 리뷰 스타일을 적어주세요 
•
민찬기
◦
역할, 책임이 잘 분리되었는지
◦
가독성
◦
변수명과 클래스명 설정이 잘 되었는지
◦
디렉토리 구조가 직관적인지
•
조홍식
◦
변수명이 누구나 쉽게 추측이 가능한지
◦
깃 컨벤션을 잘 고수했는지
◦
단일책임원칙
◦
의존 관계를 잘 분리하였는지
◦
이슈를 파서 진행한 후 해당 pr에 댓글 남기는 형식으로 진행했습니다!
•
오현직
◦
변수, 클래스, 메소드 등 코드 컨벤션이 잘 지켜졌는지git issue 작성 시, 요구사항에 맞춰 제대로 작성이 되어있는지.
◦
디렉토리 구조가 도메인 별로 분리 되어서 설계했는지
◦
commit시 , 버그, 리팩토리, 설정파일 등 어떤 작업을 했는지 확인이 가능한지
◦
그 밖에 이해가 안되는 부분 있으면 직접 Pull 해서 디버깅으로 흐름도 파악
•
박정원
◦
변수, 메소드, 클래스명이 적절한지
◦
코드 상의 중복되는 부분을 메서드로 추출하여 최대한 방지했는지
◦
코드의 주석이 적절한지
◦
로직상에서 놓친 예외처리는 없는지
•
서진수
◦
구조에 맞게 잘 짜여졌는지
◦
변수, 클래스 명 등 직관적으로 잘 짜여졌는지
◦
예외처리는 잘 되었는지
•
김세은
◦
변수명, 메소드명, 클래스명이 직관적으로 이해가 되는지
◦
코드 구조가 일관되게 짜여졌는지
◦
적절한 브랜치에 commit을 하고 commit message를 규칙에 따라 작성하는지?
•
손정아
◦
변수명이 적절하고 직관적으로 짜여져 있는지
◦
코드에 대한 주석을 잘 달았는지
◦
필요하지 않은 코드가 있는지
◦
예외처리가 잘 되어있는지
◦
같은 기능을 더 단순하게 구현할 수 있는지?
•
이지원
◦
코드 컨벤션을 잘지켰는지
◦
커밋 메시지를 잘 작성하기
◦
주석에서 코드가 하는 일 보다 코드에 담긴 의도를 설명하기
◦
중복되는 코드는 없는지
◦
작은 PR 만들기
우리 팀의 코드 리뷰 가이드를 세워봅시다!
필수)
변수명, 메서드명, 클래스명
가독성이 괜찮은 지 (별도의 컨벤션 X)
디렉토리 구조가 적절한 지
•
계층형, 도메인형은 신경쓰지 않는다
단위 commit의 크기가 적절한 지
주석이 적절하게 사용되었는 지
•
사용이 되었다면, 필요한 것인지
•
주석이 필요한데 없는지
예외처리
•
Critical한 예외가 처리 되지 않았는 지(NPE, NoSuchElementException)
옵션)
단일책임원칙
branch가 적절히 분리 되었는 지
로직
•
중복이 없는 지
•
효율성이 개선될 수 있는 지
우리 팀의 1인당 코드 리뷰 최소 시간은?
오전 10:00 ~ 오후 16:00 - 각자 pr에 대한 코드리뷰
오후 16:00 ~ 오후 18:00 - 리뷰에 대한 토의 진행
PR 방법
원활한 review를 진행하기 위해 다음과 같이 브랜치를 설정합니다.
1.
main(다른 이름도 좋습니다): 개발 진행 통합
2.
review(리뷰를 위한 브랜치)
•
해당 브랜치는 전 주 미션 내용이 남아있습니다
•
수요일 6시 이후, main → review로 pr을 진행합니다
•
이전 주 내용과 이번 주 내용의 차이를 확인할 수 있습니다
코드 리뷰 완료 확인은 어떻게 진행할까요?
•
pr review 방식
◦
Finish your review에서 전체적으로 좋았던 점, 개선할 점 남기기
- 좋았던 점
- 이러쿵
- 저러쿵
- 개선사항
- 이러쿵
- 저러쿵
Plain Text
복사
•
리뷰 완료 이후 discord 스레드에 메시지 남기기
