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

코드 리뷰 적응기

코드 리뷰 적응기

SOCAR

May 07, 2020
Tweet

More Decks by SOCAR

Other Decks in Programming

Transcript

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

    작업은 하나의 PR에서 처리하지 말고, 별개의 PR로 올린다. - 보일러 플레이트 생성 코드는 별도의 브랜치로 날리고, 해당 브랜치을 베이스로 PR을 올린다.
  2. ௏ܻ٘࠭ܳాҗೞӝਤ೧ࢲj PR 올릴 때 코멘트에 작업 의도를 남기자. - PR

    코멘트에 Markdown 이용 - 코드 코멘트 - 기획 문서, 디자인 링크 공유하기
  3. 코드 작업, 코드 리뷰… 정신없다. ੉ઁ աب௏ܻ٘࠭ܳࠅٸ 코드 리뷰 안

    되면, 릴리즈가 느려진다. 코드가 Merge 되기 위해서, 2명 이상의 reviewer의 approve가 필요하다.
  4. ௼܁੊झబ࣌ 0DUPUSFF 기존 깃 허브는 패키지 구조로 어떤 변화가 있는지

    알기 어렵다. 유료 기능은 더 좋은게 많다고 합니다…
  5. प೷੸౱ӏ஗੿ೞӝ - 최대 허용 PR 10개 - 일주일에 한 번은

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