ode Review是一個(gè)老生常談的話題缎浇,其作用及意義此處不多說(shuō)扎拣,本文僅結(jié)合我過(guò)去的工作經(jīng)歷赴肚,總結(jié)下如何在實(shí)際工作中更有效的做起來(lái)CR素跺。以下是幾點(diǎn)思考,請(qǐng)大家批評(píng)指正誉券。
一指厌、需要工具的支撐。拉幾個(gè)人坐一塊講講代碼踊跟,不是我理解的CR踩验,只能算是代碼講解。我認(rèn)為一次有效CR需要工具的支持商玫,比如開(kāi)源Review Board或者收費(fèi)code collaborator等箕憾。
二、參與人員角色拳昌。參考cc中author\reviewer\observer的劃分袭异,我認(rèn)為在實(shí)際執(zhí)行中可以這么做:author即一次CR的發(fā)起者,選同組的兩名開(kāi)發(fā)人員作為reviewer炬藤,選測(cè)試人員御铃、產(chǎn)品和研發(fā)負(fù)責(zé)人作為observer。reviewer必須要做出響應(yīng)沈矿,observer可選上真。
三、一些基本的規(guī)則羹膳。
3.1 reviewer平均每分鐘review的行數(shù)不能超過(guò)XX行
3.2 author提供必要的diff文件
3.3 一次CR不超過(guò)XX個(gè)文件
3.4 如果是修bug睡互,那一次CR盡量控制在3個(gè)bug以內(nèi)
3.5 如果是新的feature,那一次CR就對(duì)應(yīng)一個(gè)feature
3.6 CR包括但不限于代碼功能和規(guī)范
如果把CR的范圍擴(kuò)大一下到Peer Review陵像,那么需求湃缎、設(shè)計(jì)、開(kāi)發(fā)蠢壹、測(cè)試等相關(guān)的doc和plan都可以參考其中的一些規(guī)則嗓违。