///
Search
👍🏻

7조

Created
2022/10/13 06:05
Tags

팀원 각자의 코드 리뷰 스타일을 적어주세요

도성구 : 다른 분들의 코드를 읽고 배운점을 간단하게 적습니다. 또한, 다른분들께서 물어보셨던 부분들은 참고자료나 로직을 어떻게 구현했는지에 대해서 링크나 설명을 첨부합니다.
한승연 : working 브랜치에서 개발을 진행하고 이번주에 개발했던 내용을 요약해서 main 브랜치로 pr을 날립니다. 각 코드 부분에 궁금한점(질문), 좋았던 점, 의견 제시 등 자유롭게 리뷰를 남기고 보이스 채널에서 모여서 팀원들 리뷰를 진행했습니다. 이후 리뷰에서 나왔던 질문은 구글링을 하거나 강사님 질의응답 시간을 활용해 해결했습니다. (아래 링크는 현재 스터디에서 진행했던 코드 리뷰예시입니다)
9
pull
주민지 : 코드리뷰 체크리스트(ex.네이밍, 레이어 설계, 복잡성, 모듈화 등)를 공유하여 진행했었습니다.
임소망 : 각자 브랜치를 만들어서 PR을 남기고, 그 PR을 보고 리뷰 남기는 방식으로 진행했습니다. PR에서 리뷰를 남길 때는 네이밍, 주석, 로직과 추가적으로 얘기하고 싶은 부분에 대한 코멘트를 작성했습니다.
김동현2 : 사소한 오탈자보다는 설계와 어긋나는 부분이 없는지, 코드를 확장할 때 작성한 코드가 어떤 문제점이 될 수 있는 지를 생각합니다.
강민성 : 다른 분의 코드를 보고 코드 라인별로 궁금한 점이나 배운 점 혹은 좀 더 개선할 수 있을 거 같은 부분에 대해서 코드 리뷰를 했었습니다. 필요한 경우 링크를 첨부했었습니다!
권종석 : 본인 코드에서 추가로 개선할 수 있는 점들을 피드백하고, 어떻게 하는지 코드를 알려주기 보다 생각을 넓힐 수 있게 도움주면 좋겠다고 생각합니다.
지남은: 다른 분들은 코드를 어떤 식으로 진행하였는지 설계 방향을 보며 진행 상황에 맞게 가고있는지 확인한 뒤에 commit을 남겼습니다.

우리 팀의 코드 리뷰 가이드를 세워봅시다!

오전 10시 ~ 오후 6시까지 파송송계란탁 음성채널을 사용합니다.
궁금한 점이 있는 코드는 스크랩해서 리뷰를 남기고 리뷰에 대한 답글을 밑에 남기도록 합니다.
또한, 마지막에는 한줄평을 남기도록 합니다.
오후 3시 이전까지는 팀원들의 코드를 읽고 코멘트를 남기도록 합니다.
오후 3시부터는 각자 코드를 발표하는 시간을 가집니다.

Check List

구현한 로직상에서 오류가 날 수 있는 부분은 없는지
변수, 클래스, 메소드 등에 명확한 이름을 선택했는지
주석이 명확하고 유용한지
주석이 없어도 네이밍이 명확한데 필요 없는 주석이 있는지
코드가 필요 이상으로 복잡하지 않은지
스타일 가이드를 잘 지켰는지 (이 부분은 팀 내의 코딩 컨벤션을 따릅니다.)

피어리뷰 필수 항목

가장 이슈가 되는 부분 (기능 구현에 대한 부분) 언급
코드 상 이해하지 못한 부분, 문제가 되는 부분을 구체적으로 언급
보완해야할 부분에 대한 의견 제시
총평
참고자료

우리 팀의 1인당 코드 리뷰 최소 시간은?

1인당 코드리뷰 시간은 30~40분 정도 소요하도록 합니다.
오후 3시 이후에는 개별적으로 코드를 발표하고 Q&A 시간을 갖도록 합니다.

코드 리뷰 완료 확인은 어떻게 진행할까요?

오후 3시 회의시간 이전에 현재까지 리뷰한 팀원들의 이름(OO님)을 포함하여서 코드 리뷰 완료했다는 메세지를 디스코드 7조 피어리뷰 채팅방에 남기도록 합니다.
메세지 예시 : OO님, OO님 코드 리뷰 완료하였습니다. 확인 부탁드립니다!