깃헙으로 코드리뷰 하기

4,805 views
4,430 views

Published on

2012.11.01 랩 밋업데이 발표자료.

0 Comments
51 Likes
Statistics
Notes
  • Be the first to comment

No Downloads
Views
Total views
4,805
On SlideShare
0
From Embeds
0
Number of Embeds
1
Actions
Shares
0
Downloads
59
Comments
0
Likes
51
Embeds 0
No embeds

No notes for slide

깃헙으로 코드리뷰 하기

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

×