環(huán)球關(guān)注:Linus Torvals 發(fā)火:Linux PR 請(qǐng)求如果沒有注釋,不要提交過來
IT之家 2 月 22 日消息,Linux 6.3 合并窗口期已經(jīng)于近日啟動(dòng),Linus Torvalds 收到了大量的 pull request(PR)請(qǐng)求,整體而言本次窗口期的各項(xiàng)工作開展順利。
Linus Torvalds 也針對(duì)本次合并窗口期出現(xiàn)的問題發(fā)表了他的看法,IT之家翻譯如下:
(相關(guān)資料圖)
我收到了這樣一條 PR 請(qǐng)求(5c0f220e1b2d):“將‘for-linus / hardening’分支合并到‘for-next / hardening’”。
整個(gè)請(qǐng)求信息只有這一行短短的文字。
我之前已經(jīng)多次強(qiáng)調(diào),現(xiàn)在顯然需要再次強(qiáng)調(diào):如果你懶得解釋為什么需要合并,那么這條合并請(qǐng)求就應(yīng)該拋棄。
這確實(shí)應(yīng)該是每個(gè)開發(fā)人員都應(yīng)該牢記的規(guī)則,而不只是個(gè)空頭文字。
我再次強(qiáng)調(diào)一遍:如果你無法為合并請(qǐng)求提供合理的解釋或者說明,那么請(qǐng)不要提交。
真的就這么簡(jiǎn)單。在不解釋為什么存在這些合并的情況下,絕對(duì) * 永遠(yuǎn) * 沒有合并的借口。
在這種情況下,我真的認(rèn)為如果合并請(qǐng)求都沒有注釋,那么合并還有什么意義!
如果你覺得有理由的話,請(qǐng)說出來!讓合并提交看起來更加合理。
因?yàn)楝F(xiàn)在它看起來完全沒有意義。我真的 * 厭惡 * 毫無意義的合并。它們只會(huì)讓歷史看起來更糟,更難讀懂。
標(biāo)簽: Linux