깃헙으로 코드리뷰 하기
Upcoming SlideShare
Loading in...5
×
 

Like this? Share it with your network

Share

깃헙으로 코드리뷰 하기

on

  • 2,491 views

2012.11.01 랩 밋업데이 발표자료.

2012.11.01 랩 밋업데이 발표자료.

Statistics

Views

Total Views
2,491
Views on SlideShare
2,491
Embed Views
0

Actions

Likes
11
Downloads
27
Comments
0

0 Embeds 0

No embeds

Accessibility

Categories

Upload Details

Uploaded via as Adobe PDF

Usage Rights

© All Rights Reserved

Report content

Flagged as inappropriate Flag as inappropriate
Flag as inappropriate

Select your reason for flagging this presentation as inappropriate.

Cancel
  • Full Name Full Name Comment goes here.
    Are you sure you want to
    Your message goes here
    Processing…
Post Comment
Edit your comment

깃헙으로 코드리뷰 하기 Presentation Transcript

  • 1. 깃헙으로 코드리뷰 하기 포털Ajax팀 안오균
  • 2. 코드리뷰 미팅
  • 3. 우리 팀에서는. 1. 리뷰받을 사람이 미팅을 요청한다. 2. 회의실에 모여서, 작성한 코드를 설명한다. 3. 설명 중갂 중갂 질문하고 의견을 준다.
  • 4. 아쉬운 점. 1. 빠지지 않는 컨벤션 이야기. 2. 가끔은 주제를 벗어나기도. 3. 아무 말도 하지 않는 사람. 4. 코드 찾기 대화. "잠깐 잠깐, 저기요. 아뇨. 그 아래. 거기요."
  • 5. 아쉬운 점 베스트! 리뷰해줄 수 있는 범위의 한계. - 리뷰 들어가기 젂에 코드를 보지 않음. - 코드에 대해 설명해도 이해가 안됨.
  • 6. + 라인 단위 리뷰. - 코드리뷰 미팅에서는, 여러 사람의 스타일이나 의견을 들어볼 수 있다. - 코드의 상세한 부분은, 라인 단위 리뷰를 해주면 더 좋다!
  • 7. 라인 단위 리뷰를 제공하는 툴
  • 8. 툴이 없어서... ( ... ) function isTimeToGoHome(time) { if (time.isHour(7)) { // [리뷰] // 여기서는 5분 젂부터 검사하는게 어떨까요? return true; } return false; } ( ... )
  • 9. 깃헙을 이용하면 굿!
  • 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
  • 18. Pull Requests
  • 19. Discussion
  • 20. 그럼 코드리뷰는 어떻게? 1. Pull Requests 활용하기 2. Commits 로그 활용하기 위 메뉴에서는, 라인 단위로 댓글을 다는 기능을 지원합니다.
  • 21. 회원가입부터 리뷰까지!5분 만에 끝내기! 실습 고고!!
  • 22. 자세한 내용은, using-github-for-code-reviews.git