깃헙으로 코드리뷰 하기

포털Ajax팀

안오균

코드리뷰 미팅

우리 팀에서는.

1. 리뷰받을 사람이 미팅을 요청한다.

2. 회의실에 모여서, 작성한 코드를 설명한다.

3. 설명 중갂 중갂 질문하고 의견을 준다.

아쉬운 점.

1. 빠지지 않는 컨벤션 이야기.

2. 가끔은 주제를 벗어나기도.

3. 아무 말도 하지 않는 사람.

4. 코드 찾기 대화.

"잠깐 잠깐, 저기요. 아뇨. 그 아래. 거기요."

아쉬운 점 베스트!

리뷰해줄 수 있는 범위의 한계.

- 리뷰 들어가기 젂에 코드를 보지 않음.

- 코드에 대해 설명해도 이해가 안됨.

+ 라인 단위 리뷰.

- 코드리뷰 미팅에서는,

여러 사람의 스타일이나 의견을 들어볼 수 있다.

- 코드의 상세한 부분은,

라인 단위 리뷰를 해주면 더 좋다!

라인 단위 리뷰를 제공하는 툴

툴이 없어서...

( ... )

function isTimeToGoHome(time) { if (time.isHour(7)) {

// [리뷰]

// 여기서는 5분 젂부터 검사하는게 어떨까요?

return true;

}

return false;

}

( ... )

깃헙을 이용하면 굿!

Git & GitHub

Git : 분산 버젂 관리 시스템

GitHub : Git을 이용한 온라인 프로젝트 호스팅

잘 이해가 되지 않는다면,

온라인에 있는 SVN이라고 생각하시면 되요.

소셜 코딩.

- 깃헙이 대세~

- 오픈 소스 프로젝트의 대부분이 깃헙에서 짂행

- Fork & Pull Requests 모델.

(복사 & 머지 요청)과 비슷.

Fork & Pull Requests (1/6)

github local

/jquery/jquery

github에 호스팅하고 있는, 원본 프로젝트

Fork & Pull Requests (2/6)

github local

/jquery/jquery

유저 or 조직 프로젝트명

Fork & Pull Requests (3/6)

github local

/jquery/jquery

/ohgyun/jquery

Fork = 복사

Fork & Pull Requests (4/6)

github local

/jquery/jquery

/ohgyun/jquery local/ohgyun/jquery

Pull

Push

개발은 로컬에서

Commit

Fork & Pull Requests (5/6)

github local

/jquery/jquery

/ohgyun/jquery

Pull Requests

local/ohgyun/jquery

요고 내가 개발했는데, 함 보고 적용해줘~

Fork & Pull Requests (6/6)

github local

/jquery/jquery

/ohgyun/jquery

Merge!

local/ohgyun/jquery

Contributor가 됐다.

Pull Requests

Discussion

그럼 코드리뷰는 어떻게?

1. Pull Requests 활용하기

2. Commits 로그 활용하기

위 메뉴에서는,

라인 단위로 댓글을 다는 기능을 지원합니다.

회원가입부터 리뷰까지! 5분 만에 끝내기!

실습 고고!!

자세한 내용은,

using-github-for-code-reviews.git