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

코드 리뷰 적응기

Sponsored · Your Podcast. Everywhere. Effortlessly. Share. Educate. Inspire. Entertain. You do you. We'll handle the rest.

코드 리뷰 적응기

Avatar for SOCAR

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주 이상 묶여있을 시 작업 중지 - 슬랙 봇으로 온라인 리뷰를 안 한 사람에게 알람 보내기