高洛峰2017-05-02 09:51:24
Code Review, 指代碼審閱；場景：新人或初級工程師，做了一些改動，但是需要在老程式設計師的確認後再添加至主分支或特性分支；不可能讓老程式設計師坐你旁邊看程式碼嗎，這樣太浪費時間了；
舉例：
A計畫有兩個分支master分支和dev分支；现在公司在开发，所有人完成任务，都应该把分支合并至dev分支；
張三在自己的本地開發環境，基於dev分支，创建zhang_san分支；花了三天，完成一個功能；
老程式設計師覺得張三是新手，不放心就這麼將他的改動直接放進dev分支；就讓張三提個Merge申請，讓他dev分支；就让张三提个Merge申请，让他Code Review ;
於是張三就將自己的zhang_san分支提到远程服务器上，操作“New Merge Request”，发起一个Merge请求，将zhang_san合并至dev分支，指定審閱人是老程式設計師；
老程式設計師找了個空閒時間，查看這個Merge Request,覺得有點問題, 於是就提了一些建議（提的建議會自動發送到相關人的關聯郵箱）；
張三後來透過郵箱查看到老程式設計師提的建議，在自己本地zhang_san进行了修改，并git push至远程zhang_san分支；
這時的「Merge Request」會更新最新的Commit；老程序员再看这次总的改动，发现不错，挺好的，点击确认请求后，就成功将zhang_san分支上的改动合并至dev分支啦；【到這就完成了Code Review】
張三通過郵件，知道自己的分支已經被合併進開發分支dev很開心；可以刪除這個本地zhang_san分支和遠端zhang_san分支，保持分支乾淨；也可以繼續在zhang_san分支做新的開發，提交新的Merge_Request,進行新一輪的dev很开心；可以删除这个本地zhang_san分支和远程zhang_san分支，保持分支干净；也可以继续在zhang_san分支做新的开发，提交新的Merge_Request,进行新一轮的Code Review。
