clean code
1.google author guide - to getting through code review
CL (ChangeList)버전 제어에 제출되었거나 코드 검토 중인 하나의 자체 포함된 변경사항을 의미합니다.다른 조직에서는 이를 종종 "pull-request"이라고 부릅니다.gerrit코드 리뷰 + git 저장소 시스템특징리뷰 통과 조건 설정IDE 도구와 연동jen
2.CL 과 gerrit 이란?
버전 제어에 제출되었거나 코드 검토 중인 하나의 자체 포함된 변경사항을 의미합니다.다른 조직에서는 이를 종종 "pull-request"이라고 부릅니다.코드 리뷰 + git 저장소 시스템특징리뷰 통과 조건 설정IDE 도구와 연동jenkins와 같은 CI(Continuou
3.google code review(1) - overview
코드 리뷰 프로세스 기초작성자던 reviewer이던, 조금 막히면 직접 찾아가서 이야기하자. 그게 훨씬 효율적이렇게 합의를 봤고, 결론에 도달한 이유를 반드시 적어놓아야 한다.코드 리뷰는 어느 정도까지?케이스 1: 대규모 서비스 or 서비스 준비 중 or 큰 과제 or
4.google code review(2) - 구체적 방식
CL 에 대한 설명이 잘 되어있는지 가장 먼저 확인하자.step 1: 변화의 broad view를 보자.애초에 이러한 변경이 발생하지 말았어야 하는 경우에는, 변경이 발생하지 않아야 하는 이유를 즉시 설명하여 author 에게 전덜하라.이와 같은 변경을 거부할 때 개발
5.git subtree
상위 저장소는 서브모듈의 정보 중에서 checkout한 커멧의 SHA 값만 기록한다. 그러므로 상위 저장소 내 서브모듈로 들어가서 변경을 쌓아 push를 하면, conflict가 나는 경우가 많다.즉, 서브모듈을 변경하고 싶으면, 꼭 그 래포로 가서 수정해야하는 번거로
6.git hook (pre-commit 등)
https://techblog.woowahan.com/2530/https://git-scm.com/book/ko/v2/Git%EB%A7%9E%EC%B6%A4-Git-Hooksgit hook: 특정 상황(아래 사진 10가지 case)에 특정 스크립트를
7.git branch protection rule
이 옵션을 선택하면, 모든 커밋을 보호되지 않는 branch로 만들고 꺼내기 요청을 통해 제출해야 이 규칙과 일치하는 branch로 병합할 수 있습니다.branch를 이 규칙과 일치하는 branch로 병합하기 전에 통과해야 하는 상태 검사를 선택하십시오.이 옵션을 선택