Bitcoin-Core-Development

在合併之前對每個拉取請求進行多少測試?

  • December 30, 2020

通常在一個拉取請求合併後,多個其他拉取請求會跟進以清理第一個 PR 造成的問題

最近的例子: https ://github.com/bitcoin/bitcoin/pull/7957

<https://github.com/bitcoin/bitcoin/pull/8164> <https://github.com/bitcoin/bitcoin/pull/8171>

通常不會造成真正的傷害。這沒什麼大不了的,還是表明在合併比特幣拉取請求之前應該進行更多測試?

每個拉取請求通常都不是那麼大,因此查看每一行程式碼並不難。每次拉取都經過細緻的測試,以防出現任何惡意程式碼或意外缺陷。為什麼你認為在拉取之後通常需要一個多月才能被添加,即使評論中有很多討論?

我也見過很多。這是一個非常難以回答的問題,因為它實際上取決於合併問題的具體情況和嚴重程度。

就個人而言,我認為拉取請求不應該像有時那樣快速合併,因為應該有時間來審查即使是微小的變化。維護者處於一個棘手的位置,儘管有時拉取請求沒有得到審查,開放數週或數月並且過時(需要變基)。出於充分的理由,如果維護者認為它們的風險低,有時他們會合併拉取請求而很少進行集體審查。

在理想的世界中,每個拉取請求都會立即進行測試和審查,然後維護人員不必對低風險和不需要太多集體審查做出判斷。但是我們並不是在一個理想的世界裡,而且總的來說(在我的無保留意見中)我認為維護者的這些判斷是正確的。

最重要的事情顯然是避免在發佈時出現問題。如果問題在發布之前在額外的拉取請求中得到解決,那麼問題相對較小。

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