Contenu connexe
Similaire à 깃헙으로 코드리뷰 하기 (20)
깃헙으로 코드리뷰 하기
- 3. 우리 팀에서는.
1. 리뷰받을 사람이 미팅을 요청한다.
2. 회의실에 모여서, 작성한 코드를 설명한다.
3. 설명 중갂 중갂 질문하고 의견을 준다.
- 4. 아쉬운 점.
1. 빠지지 않는 컨벤션 이야기.
2. 가끔은 주제를 벗어나기도.
3. 아무 말도 하지 않는 사람.
4. 코드 찾기 대화.
"잠깐 잠깐, 저기요. 아뇨. 그 아래. 거기요."
- 5. 아쉬운 점 베스트!
리뷰해줄 수 있는 범위의 한계.
- 리뷰 들어가기 젂에 코드를 보지 않음.
- 코드에 대해 설명해도 이해가 안됨.
- 6. + 라인 단위 리뷰.
- 코드리뷰 미팅에서는,
여러 사람의 스타일이나 의견을 들어볼 수 있다.
- 코드의 상세한 부분은,
라인 단위 리뷰를 해주면 더 좋다!
- 8. 툴이 없어서...
( ... )
function isTimeToGoHome(time) {
if (time.isHour(7)) {
// [리뷰]
// 여기서는 5분 젂부터 검사하는게 어떨까요?
return true;
}
return false;
}
( ... )
- 10. Git & GitHub
Git : 분산 버젂 관리 시스템
GitHub : Git을 이용한 온라인 프로젝트 호스팅
잘 이해가 되지 않는다면,
온라인에 있는 SVN이라고 생각하시면 되요.
- 11. 소셜 코딩.
- 깃헙이 대세~
- 오픈 소스 프로젝트의 대부분이 깃헙에서 짂행
- Fork & Pull Requests 모델.
(복사 & 머지 요청)과 비슷.
- 12. Fork & Pull Requests (1/6)
github local
/jquery/jquery
github에 호스팅하고 있는,
원본 프로젝트
- 13. Fork & Pull Requests (2/6)
github local
/jquery/jquery
유저 or 조직
프로젝트명
- 14. Fork & Pull Requests (3/6)
github local
/jquery/jquery
Fork = 복사
/ohgyun/jquery
- 15. Fork & Pull Requests (4/6)
github local
/jquery/jquery
개발은 로컬에서
Commit
Pull
/ohgyun/jquery local/ohgyun/jquery
Push
- 16. Fork & Pull Requests (5/6)
github local
/jquery/jquery
Pull Requests
요고 내가 개발했는데,
함 보고 적용해줘~
/ohgyun/jquery local/ohgyun/jquery
- 17. Fork & Pull Requests (6/6)
github local
/jquery/jquery
Merge!
Contributor가 됐다.
/ohgyun/jquery local/ohgyun/jquery
- 20. 그럼 코드리뷰는 어떻게?
1. Pull Requests 활용하기
2. Commits 로그 활용하기
위 메뉴에서는,
라인 단위로 댓글을 다는 기능을 지원합니다.