<div id="readability-page-1" class="page"><div id="wrap">

        <main>

          
          <div><p data-ke-size="size16"><a href="https://www.youtube.com/watch?v=ssDMIcPBqUE&amp;ab_channel=%EC%9A%B0%EC%95%84%ED%95%9CTech">https://www.youtube.com/watch?v=ssDMIcPBqUE&amp;ab_channel=%EC%9A%B0%EC%95%84%ED%95%9CTech</a></p>
<figure data-ke-type="video" data-ke-style="alignCenter" data-video-host="youtube" data-video-url="https://www.youtube.com/watch?v=ssDMIcPBqUE" data-video-thumbnail="https://scrap.kakaocdn.net/dn/b2QR51/hyOcuc7ULU/vyVsjFqHQSNPkN5pnORliK/img.jpg?width=480&amp;height=360&amp;face=324_95_389_166" data-video-width="480" data-video-height="360" data-video-origin-width="480" data-video-origin-height="360" data-ke-mobilestyle="widthContent"><iframe src="https://www.youtube.com/embed/ssDMIcPBqUE" width="480" height="360" frameborder="0" allowfullscreen="true"></iframe>
<figcaption></figcaption>
<img src="https://scrap.kakaocdn.net/dn/b2QR51/hyOcuc7ULU/vyVsjFqHQSNPkN5pnORliK/img.jpg?width=480&amp;height=360&amp;face=324_95_389_166"></figure>
<p data-ke-size="size16">알게 된 부분</p>
<ol data-ke-list-type="disc">
<li>unused import, declaration =&gt; error로 표시 되도록 설정</li>
<li>PR을 올릴 때 주석 달기 -&gt; 먼저 읽어보고 설명을 커멘트로 남겨서 리뷰어의 시간 절약</li>
<li>리뷰어를 모두 포함 시켜라</li>
<li>의미 있는 커밋으로 분리
<ol data-ke-list-type="disc">
<li>파일 생성부터 커밋을 분리</li>
<li>테스트 추가 커밋 등</li>
</ol>
</li>
<li>리뷰에 대한 룰 확보
<ol data-ke-list-type="disc">
<li>아침 30분, 점심 이후 30분</li>
<li>PR의 변경분이 적도록 노력</li>
<li>리뷰하는 것 자체의 노고를 인정 해줘야 함.</li>
</ol>
</li>
<li>리뷰의 핵심
<ol data-ke-list-type="disc">
<li>무엇이 코드를 나아지게 하는가? 에 초점</li>
<li>누가 그런 아이디어를 냈는지가 아니다.</li>
<li>너만 빼라. 너라고 하지 않는게 좋고, 대상은 코드 그 자체다 사람이 아님.</li>
<li>I message 대화법:</li>
<li>~ 하는 것을 제안 합니다. ~ 하는게 어떨까요?
<ol data-ke-list-type="disc">
<li>리뷰대로 해도 되고, 안해도 되고 자유</li>
<li>오픈 커뮤니케이션을 유지 할 것</li>
</ol>
</li>
<li>건설적인 피드백이 목적이지, 건설적으로 피드백을 줄 것이 아니라면 의미 없음.</li>
<li>진정한 칭찬을 써주어라
<ol data-ke-list-type="disc">
<li>칭찬 해야 할 부분을 지정해서 쓸 것!</li>
</ol>
</li>
<li>의견을 줄때, 제안하는 변경과 변경의 이유를 모두 설명하라</li>
<li>반복적인 실수 패턴에 대해서는 2~3개 정도의 예를 언급하라 - 모든 경우를 다 말할 필요 없다.</li>
<li>교착 상태를 적극적으로 처리 필요
<ol data-ke-list-type="disc">
<li>만나서 이야기 하거나</li>
<li>설계 리뷰를 고려 하거나</li>
<li>아주 심각하지 않다면, 인정하고 협업 관계를 유지 하거나</li>
</ol>
</li>
<li>짝 프로그래밍을 하면서 어떻게 고치는 게 좋은지 보여주고
<ol data-ke-list-type="disc">
<li>20분 동안 페어 프로그래밍, 2시간 스스로 개선하도록 시간 부여</li>
</ol>
</li>
<li>결정은 리뷰 요청자가 하는 것
<ol data-ke-list-type="disc">
<li>할 수 있는 최고의 설계를 추구 하는 것</li>
<li>불완전한 해결책도 받아들이는 것도 좋음</li>
<li>모든 설계 결함이 꼭 장애가 되진 않음</li>
<li>코드 자체가 이상하게 그날 이상 할 수도 있음. -&gt; 저자에게 개발 외적인 문제가 생겼을 수도 있다.<br>QA) 리뷰어의 응답을 기다리다가 지체 되는 현상이 발생할 경우?</li>
</ol>
</li>
</ol>
</li>
</ol>
<ul data-ke-list-type="disc">
<li>팀에서 룰을 정할 수 있다.
<ul data-ke-list-type="disc">
<li>반나절동안 리뷰가 안된다면, 리뷰어 승인 없이 배포 나가고 스스로 책임을 진다. (좋은 방법 같다.)</li>
</ul>
</li>
</ul>            
</div>
          

          
          
          

        </main>
        
      </div></div>

Seungdols Company

seungdols

https://www.youtube.com/watch?v=ssDMIcPBqUE&ab_channel=%EC%9A%B0%EC%95%84%ED%95%9CTech 알게 된 부분 unused import, declaration => error로 표시 되도록 설정 PR을 올릴 때 주석 달기 -> 먼저 읽어보고 설명을 커멘트로 남겨서 리뷰어의 시간 절약 리뷰어를 모두 포함 시켜라 의미 있는 커밋으로 분리 파일 생성부터 커밋을 분리 테스트 추가 커밋 등 리뷰에 대..

우아한테크 세미나 - 지속가능한 SW 개발을 위한 코드리뷰

댓글