【播资讯】Linus Torvalds 怒怼:不要提交没有注释的请求

2023-02-24 08:14:41 来源:CSDN
整理 |禾木木 出品 | CSDN(ID:CSDNnews)

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/

标签: 开发人员 在不解释 毫无意义

珠宝展示