V8 提交者和審查者的責任
當您提交到 V8 儲存庫時,請務必遵循這些準則(改編自 https://dev.chromium.org/developers/committers-responsibility)
- 為您的變更和您被要求審查的程式碼修補程式找到合適的審查者。
- 在您發布變更之前和之後,請在即時通訊和/或電子郵件中保持聯繫。
- 在您的變更後,請觀看 瀑布,直到所有機器人都變為綠色。
- 在發布 TBR 變更(待審查)時,請務必通知您正在變更其程式碼的人員。通常只需寄送審查電子郵件即可。
簡而言之,請為專案做正確的事,而不是最容易提交程式碼的事,最重要的是:運用您的最佳判斷力。
不要害怕發問。總會有人會立即閱讀寄送到 v8-committers 郵件串列的訊息,他們可以幫助您。
具有多位審查者的變更 #
偶爾會有一些變更有很多審查者,因為有時由於多個責任和專業領域,可能需要多個人了解變更。
問題在於,如果沒有某些準則,這些審查中就不會給出明確的責任。
如果您是變更的唯一審查者,您知道您必須做好工作。當有其他三個人時,您有時會假設其他人一定仔細查看了審查的某一部分。有時所有審查者都這麼想,而變更沒有得到適當審查。
在其他情況下,一些審查者對程式碼修補程式說「LGTM」,而另一些審查者仍在期待變更。作者可能會對審查狀態感到困惑,並且在至少一位審查者在提交前期待進一步變更的情況下,已簽入一些程式碼修補程式。
同時,我們希望鼓勵許多人參與審查流程並關注正在發生的事情。
因此,以下是幫助釐清流程的一些準則
當程式碼修補程式作者要求多位審查者時,他們應在審查要求電子郵件中明確說明他們期望每位審查者的責任是什麼。例如,您可以在電子郵件中寫下這段文字
- larry: bitmap changes - sergey: process hacks - everybody else: FYI
在這種情況下,您可能在審查清單中,是因為您要求了解多處理程序變更,但您不會是主要審查者,而作者和其他審查者不會期望您詳細審查所有差異。
如果您收到包含許多其他人的審查,而作者沒有執行 (1),如果您不想詳細審查所有內容,請詢問他們您負責哪一部分。
作者應等到審查者清單上的所有人核准後再簽入。
在沒有明確審查責任的審查中(例如路過審查)的人員應超級有回應,且不應延誤審查。如果他們這樣做,修補程式作者應隨時對他們進行無情的 ping。
如果您是審查中的「供參考」人員,而且您實際上沒有詳細審查(或根本沒有審查),但對修補程式沒有問題,請注意這一點。您可以說「橡皮圖章」或「確認」之類的話,而不是「LGTM」。這樣,真正的審查者就知道不要相信您已替他們完成工作,但修補程式的作者知道他們不必等待您進一步的回饋。希望我們仍然可以讓所有人了解最新進度,但擁有明確的所有權和詳細的審查。這甚至可能會加快一些變更,因為您可以快速「確認」您不在意的變更,而作者知道他們不必等待您提供回饋。