Code Review, 코드 검토를 의미합니다. 초보자 또는 주니어 엔지니어가 일부 변경을 수행하지만 이를 메인 브랜치나 기능 브랜치에 추가하기 전에 숙련된 프로그래머의 확인이 필요합니다. 숙련된 프로그래머를 옆에 두는 것은 불가능합니다. 너랑 코드를 봐? 이건 정말 시간낭비야;
예: 프로젝트에는 2개의 브랜치 master 브랜치와 dev 브랜치가 있습니다. 현재 회사는 개발 중이므로 작업을 완료할 때 모든 사람이 브랜치를 dev 브랜치에 병합해야 합니다.
Zhang San은 자신의 로컬 개발 환경에서
브랜치를 기반으로 dev 브랜치를 만들었습니다. zhang_san
늙은 프로그래머는 Zhang San이 초보자라고 생각하고 자신의 변경 사항을
브랜치에 직접 적용하는 것이 걱정되어 Zhang San에게 dev신청서를 제출하고 Merge;Code Review
그래서 Zhang San은 자신의
브랜치를 원격 서버에 언급하고 "새 병합 요청"을 실행하고 zhang_san 요청을 시작한 다음 Merge 브랜치를 zhang_san 브랜치에 병합하고 리뷰어를 이전 프로그래머로 지정했습니다 dev.
늙은 프로그래머는 여유 시간을 갖고 이것을 확인하고
뭔가 잘못되었다고 생각하여 몇 가지 제안을 했습니다(제안은 해당 사람의 관련 이메일 주소로 자동 전송됩니다 Merge Request).
Zhang San은 나중에 이전 프로그래머가 이메일을 통해 제안한 내용을 확인하고 로컬에서 수정한 다음
원격 zhang_san 지점으로 전송 git push합니다.
zhang_san이번에 "병합 요청"은 최신
으로 업데이트됩니다. 이전 프로그래머가 일반 변경 사항을 다시 살펴보고 요청을 확인하기 위해 클릭한 후
분기의 변경 사항이 매우 좋았다는 것을 알았습니다. Commit 브랜치에 성공적으로 병합되었습니다. ;【코드 리뷰가 이제 완료되었습니다】zhang_sandevZhang San은 자신의 브랜치가 개발 브랜치로 병합되었다는 소식을 듣고 매우 기뻤습니다
. 그는 로컬
브랜치와 원격 dev 브랜치를 삭제하여 브랜치를 깨끗하게 유지할 수도 있습니다. zhang_san 분기 및 새 분기 제출 zhang_san , 새로운 zhang_san 라운드를 위해. Merge_Request
Code Review
, 코드 검토를 의미합니다. 초보자 또는 주니어 엔지니어가 일부 변경을 수행하지만 이를 메인 브랜치나 기능 브랜치에 추가하기 전에 숙련된 프로그래머의 확인이 필요합니다. 숙련된 프로그래머를 옆에 두는 것은 불가능합니다. 너랑 코드를 봐? 이건 정말 시간낭비야;예:
Zhang San은 자신의 로컬 개발 환경에서프로젝트에는 2개의 브랜치
master
브랜치와dev
브랜치가 있습니다. 현재 회사는 개발 중이므로 작업을 완료할 때 모든 사람이 브랜치를dev
브랜치에 병합해야 합니다.브랜치를 기반으로
늙은 프로그래머는 Zhang San이 초보자라고 생각하고 자신의 변경 사항을dev
브랜치를 만들었습니다.zhang_san
브랜치에 직접 적용하는 것이 걱정되어 Zhang San에게
그래서 Zhang San은 자신의dev
신청서를 제출하고Merge
;Code Review
브랜치를 원격 서버에 언급하고 "새 병합 요청"을 실행하고
늙은 프로그래머는 여유 시간을 갖고 이것을 확인하고zhang_san
요청을 시작한 다음Merge
브랜치를zhang_san
브랜치에 병합하고 리뷰어를 이전 프로그래머로 지정했습니다dev
.뭔가 잘못되었다고 생각하여 몇 가지 제안을 했습니다(제안은 해당 사람의 관련 이메일 주소로 자동 전송됩니다
Zhang San은 나중에 이전 프로그래머가 이메일을 통해 제안한 내용을 확인하고 로컬에서 수정한 다음Merge Request
).원격
으로 업데이트됩니다. 이전 프로그래머가 일반 변경 사항을 다시 살펴보고 요청을 확인하기 위해 클릭한 후zhang_san
지점으로 전송git push
합니다.zhang_san
이번에 "병합 요청"은 최신분기의 변경 사항이 매우 좋았다는 것을 알았습니다.
. 그는 로컬Commit
브랜치에 성공적으로 병합되었습니다. ;【코드 리뷰가 이제 완료되었습니다】zhang_san
dev
Zhang San은 자신의 브랜치가 개발 브랜치로 병합되었다는 소식을 듣고 매우 기뻤습니다브랜치와 원격
dev
브랜치를 삭제하여 브랜치를 깨끗하게 유지할 수도 있습니다.zhang_san
분기 및 새 분기 제출zhang_san
, 새로운zhang_san
라운드를 위해.Merge_Request
코드 리뷰는 코드 리뷰입니다. Baidu에서 검색하시면 많은 글이 있습니다
당신이 작성한 코드를 다른 사람이 읽을 수 있고, 당신도 다른 사람의 코드를 읽을 수 있습니다
리뷰가 끝났고 다시 온라인에 올라올 예정입니다
코드 리뷰, 일반적으로 대기업에는 코드 리뷰가 필요합니다. 코드에 문제가 있거나 사양에 맞지 않는 경우 온라인에 접속하기 전에 올바르게 수정하세요.
3천 명의 목숨을 잃었습니다: 기존 코드의 함정