Upgrade to Pro — share decks privately, control downloads, hide ads and more …

코드 리뷰 적응기

코드 리뷰 적응기

B77ee7d45cdacb6e79d88c95822ca71e?s=128

SOCAR

May 07, 2020
Tweet

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%BSL5IFNF 다크 모드 덕후인 나는, 라이트 모드 Github가 눈

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

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

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

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

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

    활성화 더 효과적인 리뷰 환경
  29. प೷੸౱ӏ஗੿ೞӝ - 최대 허용 PR 10개 - 일주일에 한 번은

    오프라인 코드 리뷰 회의 시간 가지기 - 본인 PR 1주 이상 묶여있을 시 작업 중지 - 슬랙 봇으로 온라인 리뷰를 안 한 사람에게 알람 보내기
  30. Q&A 이메일: greedy0110@gmail.com

  31. 감사합니다.