git - code review是什么?
PHP中文网
PHP中文网 2017-05-02 09:49:24
0
5
1277

新手刚入门,能否简单讲一下code review是什么东西?怎么进行code review?
谢谢

PHP中文网
PHP中文网

认证高级PHP讲师

모든 응답(5)
小葫芦

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_san devZhang San은 자신의 브랜치가 개발 브랜치로 병합되었다는 소식을 듣고 매우 기뻤습니다

. 그는 로컬

브랜치와 원격 dev 브랜치를 삭제하여 브랜치를 깨끗하게 유지할 수도 있습니다. zhang_san 분기 및 새 분기 제출 zhang_san , 새로운 zhang_san 라운드를 위해. Merge_Request

淡淡烟草味

코드 리뷰는 코드 리뷰입니다. Baidu에서 검색하시면 많은 글이 있습니다

phpcn_u1582

당신이 작성한 코드를 다른 사람이 읽을 수 있고, 당신도 다른 사람의 코드를 읽을 수 있습니다

리뷰가 끝났고 다시 온라인에 올라올 예정입니다

过去多啦不再A梦

코드 리뷰, 일반적으로 대기업에는 코드 리뷰가 필요합니다. 코드에 문제가 있거나 사양에 맞지 않는 경우 온라인에 접속하기 전에 올바르게 수정하세요.

Ty80

3천 명의 목숨을 잃었습니다: 기존 코드의 함정

최신 다운로드
더>
웹 효과
웹사이트 소스 코드
웹사이트 자료
프론트엔드 템플릿