Bitcoin-Core-Development

我們是否應該在審查拉取請求的過程中通知其他人所有測試都已通過?

  • August 15, 2022

我不確定是否有合適的地方問這個問題,如果它離題,請告訴我。

在審查比特幣核心的拉取請求時,是否經常評論並通知其他人該 PR 的所有測試都已通過?或者我們應該只留下評論以通知失敗的測試?

如果您所做的只是在標準系統上執行自動測試,並發現它們成功了,那麼評論就毫無意義了。

持續集成系統已經執行了所有測試(在各種平台上),並且在所有測試通過之前通常不會合併拉取請求。如果您所做的只是在相當標準的系統上執行測試,那麼評論這些測試通過並不會添加任何資訊。

當然,如果你做得更多,例如:

  • 您正在使用一個不尋常的作業系統,並且 PR 以某種方式與作業系統互動,評論 PR 確實不會為您破壞事情是有用的。
  • 您通過執行 PR 的自動測試之一尚未涵蓋的操作場景(如 RPC 呼叫……)進行了手動測試,或者您正在主網上進行測試,或者在 GUI 中測試步驟(不受自動測試的影響),這也很有用。
  • 正如您在回答中已經指出的那樣,如果在 PR 之前通常不會為您失敗的測試失敗,那顯然也很有用。
  • 如果您進行了程式碼審查並報告了結果(請參閱 Michael Folkson 的回答),添加您也執行測試不會有什麼壞處。

引用自:https://bitcoin.stackexchange.com/questions/114888