Linus Torvalds 怒懟:不要提交沒有註釋的請求

整理 |禾木木

Linux 6.3 合併窗口期已經於近日開啟,Linus Torvalds 收到了大量的 pull request(PR)請求,總體來看,本次窗口期的各項工作開展順利。

但 Torvalds 覺得有必要分享一些關於 Git 合併日誌訊息的建議。因為他對部分合並請求的日誌資訊非常惱火:「如果你不能解釋這些合併請求,那麼就不要提交,這是很簡單的道理。如果不解釋提交合並請求的原因,那就是在生產沒必要的東西。」

Linus Torvalds 也針對本次合併窗口期出現的問題發表了他的看法:

我收到了這樣一條 PR 請求(5c0f220e1b2d):「將‘for-linus / hardening’分支合併到‘for-next / hardening’」。整個請求資訊只有這一行短短的文字。

我已經多次強調,顯然需要再次強調:如果你無法解釋為什麼需要合併,那麼這條合併請求就是垃圾的存在。

這確實應該是每個開發人員都應該牢記的規則,而不只是個空頭文字。

我再次強調一遍:如果你無法為合併請求提供合理的解釋或者說明,那麼請不要提交。真的就這麼簡單。在不解釋為什麼存在這些合併的情況下,絕對沒有合併的藉口。

在這種情況下,我真的認為如果合併請求都沒有註釋,那麼合併還有什麼意義!如果你覺得有理由的話,請說出來!讓合併提交看起來更加合理。

因為現在它看起來完全沒有意義。我真的厭惡毫無意義的合併。它們只會讓歷史看起來更糟,更難讀懂。

其實,這次合併是對之前的 Linux 6.2 的修復,相關開發人員表示:未來會更清楚地表達合併請求的理由。

參考連結:

https://www.phoronix.com/news/Linus-Torvalds-Git-Merge-Wisdom

https://lore.kernel.org/lkml/CAHk-=wgw++ccN-Pd1npZsBSDD3z6EGUSKsWuAEh5YC-TmfJAug@mail.gmail.com/

相關文章

Windows 7 徹底退出歷史舞臺!

Windows 7 徹底退出歷史舞臺!

你還在用 Windows 7 系統嗎? 整理 | 禾木木 今日一條 #Windows7將徹底退出歷史舞臺#的話題衝上了熱搜,引發了網友們的熱...