코드 리뷰 적응기

Transcript

1. ॑஠
   नथ޹ যࢲ৬
   ௏٘
   ܻ࠭ח
   ୊਺੉૑ ઱פয ѐߊ੗, ௏٘ ܻ࠭ ੸਽ӝ

2. য়ט
   ফӝೡ
   Ѫ 코드 리뷰 경험 없는 개발자의 코드 리뷰 적응기

3. ௏٘
   ܻ࠭о
   ޤ૑
   코드리뷰 경험 전무 개인 프로젝트 팀 프로젝트

4. ୊਺
   ௏٘
   ܻ࠭
   അ੢
   ݾب

5. 내 코드는 그냥 통과할 수 있을 것 같다는 막연한 자신감이

   듦 ୊਺
   ௏٘
   ܻ࠭
   അ੢
   ݾب

6. PR 올리기

7. ղ
   ୐
   13

8. ਋ܻ
   ౱੄
   13
   ӏ஗ 개인 브랜치 네이밍 깃허브 마일스톤 ,라벨 세팅

9. ѐੋ
   ࠳ے஖
   ֎੉߁ CFBOBXFTPNFQSFTFOUBUJPO

10. ݃ੌ
    झహ
    ۄ߰ 마일스톤 프로젝트 전체 진행 상황 라벨 PR의 성격

    명시

11. ௾
    ੘স੉
    ٜয৳׮ 이슈 할당 TF 단위 작업 TF 단위 검증 작업

    완료?

12. 도저히 코드 리뷰를 통과하지 못하고 머지되지 않음 ௾
    ੘স
    ৮ܐj 코드 리뷰까지

    끝나야 작업 끝 거의 2주 no merge

13. 코드 리뷰를 쉽게 통과하기 위해서 어떤 노력을 해야 할까?

14. ௏٘
    ܻ࠭ܳ
    ాҗೞӝ
    ਤ೧ࢲj 설계 리뷰 문서 공유 관련 코드 새로운 구조

15. ௏٘
    ܻ࠭ܳ
    ాҗೞӝ
    ਤ೧ࢲj PR을 최대한 짧게 유지한다. - 작업 중에 보이는 추가

    작업은 하나의 PR에서 처리하지 말고, 별개의 PR로 올린다. - 보일러 플레이트 생성 코드는 별도의 브랜치로 날리고, 해당 브랜치을 베이스로 PR을 올린다.

16. ௏٘
    ܻ࠭ܳ
    ాҗೞӝ
    ਤ೧ࢲj PR 올릴 때 코멘트에 작업 의도를 남기자. - PR

    코멘트에 Markdown 이용 - 코드 코멘트 - 기획 문서, 디자인 링크 공유하기

17. 참조가 될 만한 문서는 모두 링크를 건다. 세부 구현 아이디어에

    대해서 설명한다.

18. 코드 리뷰 보기

19. 코드 작업, 코드 리뷰… 정신없다. ੉ઁ
    աب
    ௏٘
    ܻ࠭ܳ
    ࠅ
    ٸ 코드 리뷰 안

    되면, 릴리즈가 느려진다. 코드가 Merge 되기 위해서, 2명 이상의 reviewer의 approve가 필요하다.

20. 코드 리뷰를 쉽게 보기 위해서 어떤 노력을 해야할까?

21. ௏٘
    ܻ࠭ܳ
    औѱ
    ࠁӝ
    ਤ೧ࢲj Github의 가독성을 높이자. (feat. Chrome Extension)

22. ௼܁
    ੊झబ࣌
    0DUPUSFF 기존 깃 허브는 패키지 구조로 어떤 변화가 있는지

    알기 어렵다. 유료 기능은 더 좋은게 많다고 합니다…

23. ௼܁
    ੊झబ࣌
    (JUIVC
    %BSL
    5IFNF 다크 모드 덕후인 나는, 라이트 모드 Github가 눈

    아프다.

24. ௏٘
    ܻ࠭ܳ
    औѱ
    ࠁӝ
    ਤ೧ࢲj 실제로 코드를 받아오자. IDE 활용하기.

25. *%&ীࢲ
    ௏٘ܳ
    ࠁݶ 코드 구조 파악 코드 구조 제안 동작성 확인

26. ௏٘
    ܻ࠭ܳ
    औѱ
    ࠁӝ
    ਤ೧ࢲj 오프라인 코드 리뷰 회의를 진행하자. 온라인을 벗어나자.

27. য়೐ۄੋ
    ௏٘
    ܻ࠭
    ݽف
    झ௼ܽ
    খী
    ݽৈࢲ
    ௏٘
    ܻ࠭ 온라인 한계 촉박한 일정 모든 팀원이 알아야

    할 중요한 기능 집중도

28. ௏٘
    ܻ࠭ח
    ౱
    ޙച׮ 매주 금요일 정기 회고를 통해 커뮤니케이션 PR 규칙 리뷰

    활성화 더 효과적인 리뷰 환경

29. प೷੸
    ౱
    ӏ஗
    ੿ೞӝ - 최대 허용 PR 10개 - 일주일에 한 번은

    오프라인 코드 리뷰 회의 시간 가지기 - 본인 PR 1주 이상 묶여있을 시 작업 중지 - 슬랙 봇으로 온라인 리뷰를 안 한 사람에게 알람 보내기

30. Q&A 이메일: greedy0110@gmail.com

31. 감사합니다.