V8 提交者與審閱者的職責
當您向 V8 存儲庫提交代碼時,請確保遵循以下指導方針(改編自 https://dev.chromium.org/developers/committers-responsibility):
- 為您的更改以及您被要求審閱的補丁尋找合適的審閱者。
- 在更改提交前後,請在即時通訊和/或電子郵件上保持聯繫。
- 監控 waterfall 直到您的更改後所有機器人呈現綠燈。
- 當提交 TBR 更改(待審閱)時,務必通知涉及代碼變動的相關人員。通常只需發送審閱電子郵件即可。
簡言之,為項目做正確的事,而不是為了提交代碼選擇最簡單的路徑,最重要的是:運用您的最佳判斷。
不要害怕提問。永遠會有人立即閱讀發送到 v8-committers 郵件列表的消息並提供幫助。
多個審閱者的更改
有時會有需要多位審閱者參與的更改,因為某些更改可能涉及多個責任領域和專業知識。
問題在於,如果沒有一些規則,這些審閱中將無法明確分配責任。
如果您是唯一的審閱者,那麼您知道自己必須認真完成工作。但如果有另外三個人參與,您可能會假設其他人已經仔細查看了某些部分。有時所有的審閱者都這樣認為,結果是更改並未被正確地審閱。
在其他情況下,有些審閱者對補丁表示“LGTM”(看起來不錯),而其他人則期待進一步的更改。此時,作者可能會對審閱的狀態感到困惑,結果導致有人期待更多更改的情況下補丁已經提交了。
同時,我們希望鼓勵更多人參與審閱過程,並關注正在發生的事情。
因此,以下是一些有助於澄清流程的指導方針:
-
當補丁作者請求多於一名審閱者時,他們應在審閱請求電子郵件中明確說明每位審閱者的責任。例如,您可以在電子郵件中這樣寫:
- larry: 位圖更改
- sergey: 流程處理技巧
- 其他人:僅供參考 -
在這種情況下,您可能會因為希望參與多進程更改的流程而在審閱列表中,但您不會是主要審閱者,作者和其他審閱者也不會期望您詳細審閱所有差異。
-
如果您收到包含許多人參與的審閱,而作者未執行第 (1) 點,請在您不想詳細審閱整體內容時向作者詢問您負責的部分。
-
作者應等待審閱列表中所有人的批准後才能提交。
-
對於在審閱中未明確分配責任(即隨機瀏覽的審閱者),應保持高度響應,並且不應阻礙審閱進度。補丁作者如果需要,可以毫不留情地催促這些審閱者。
-
如果您是審閱中的“僅供參考”人員,且您確實未詳細審閱(或根本未審閱),但對補丁並無異議,請註明這一點。您可以使用“橡皮圖章”或“ACK”這樣的詞語代替“LGTM”。這樣,真正的審閱者可以知道不應依賴於您代為完成工作,而補丁作者也知道他們不必等待您的進一步反饋。我們希望能夠讓每個人都參與其中的同時,確保有清晰的責任分配和詳細的審閱。這甚至可以加速某些更改,因為您可以快速地對那些您不關心的更改“ACK”,而作者則知道他們不必等待您的反饋。