Google 如何進行 Code Review - 3
一窺 Google 內部程式審核的指導原則 - Navigating a CL in review

原文
Navigating a CL in review (審核中該如何在 CL 巡航)
Summary (總結)
現在你已經知道審核時要看些什麼,但怎樣才是審核分散在多個文件中的改動最有效的方法?
- 改動是否合理? 他是否有好的描述 (description)
- 優先看 CL 最重要的改動。它整體是否有完善的設計?
- 用合理的順序看 CL 剩餘的改動
Step One: Take a broad view of the change (用宏觀的角度來看待改動)
Look at the CL CL description and what the CL does in general.
查看 CL 描述以及它做什麼
該改動是否有意義、合理?如果在第一時間認為不應該發生這種變化,請立即說明為什麼不該這樣做的原因。 當拒絕類似這樣的更改時,向開發人員提供建議告訴他們應該怎麼做什麼也是一個好主意 [1]。
例如,你可以說「看起來你已經完成一些不錯的事情,謝謝! 但我們實際上正朝著刪除你正在修改的 FooWidget 系統的方向前進,所以現階段我們不想對它進行任何新的修改。 你覺得不如重構我們的新 BarWidget class 的主意如何?」
值得注意的是,審核者在委婉拒絕該 CL 的同時提供替代方案
,而且仍然保持「禮貌
」。這種禮貌是很重要,因為我們希望表明即使不同意也會相互保持尊重。
如果你有幾個 CL 裡頭包含你不想這樣做的改動時 (a few CLs that represent changes you don’t want to make), 你應該考慮重新開發團隊的開發過程,或發佈開發流程給外部貢獻者 (contributor) 知道,以便在任何 CL 被撰寫前有更多的溝通。 最好是在他們開始投入大量工作前説「不」,避免那些已經投入心血的工作現在必須被拋棄或徹底重寫。
- [1] 提供替代方案讓對方知道該怎麼做,而非讓其自行獨自摸索。

點出可能問題,告知替代方案或指點方向
Step Two: Examine the main parts of the CL (檢查 CL 主要的部分)
Find the file or files that are the “main” part of this CL.
找到 CL 最核心的部分的那些文件