Linus Torvalds对Linux 6.3执行不佳的拉取请求做出反应

linus torvalds

Linux Torvalds不容忍直接阻碍Linux内核开发工作流程的错误。

例如,去年,当他宣布发布候选的Linux内核6.1,他包括了一条针对开发人员在最后一刻提交代码的严厉信息。

这一次,类似的事情再次发生。

莱纳斯建议解释合并的原因

在查看开发人员发出的请求时,Linus看到Linux Kernel 6.3的合并窗口正在打开,于是发现了一个一行提交,其中提到了“合并分支'for-linus/hardning'into to-for-Next/hardning”。

虽然请求是合并,这可能是对Linux6.2周期的修复,Linus Torvalds不高兴看到它是这样的。

linux kernel commit

这就是莱纳斯·托瓦尔兹响应拉取请求:

而这一行简短日志部分实际上就是整个提交消息。

我之前已经说过这一点,显然我需要再说一遍:如果你不能费心解释合并存在的原因,那么按照定义,合并是垃圾。

这真的应该是每个开发人员都应该牢记的规则。我并不只是把随机的单词按随机的顺序组合在一起。

我重复一遍:如果你不能解释一次合并,那么就不要去做。

真的就是这么简单。在不解释为什么会存在合并的情况下,绝对(永远不会)有合并的借口。

是啊,他真的很生气。他继续强调,这一撤军请求可能根本没有正当理由,主要是因为缺乏解释:

在这种情况下,我真的认为合并根本不应该存在,而缺乏解释是因为它*没有*解释。

但如果这是有原因的,那么只需声明它,该死的,并使合并提交看起来合理。

因为现在它看起来完全没有意义。从字面上看,我讨厌毫无意义的合并。它们只会让历史看起来更糟糕,更难读懂。

考虑到维护/开发Linux内核是软件行业最重要的任务之一,Linus Torvalds只是希望看到事情以正确的方式进行。

你能从中学到什么?:如果你是一名为Linux内核做出贡献的开发人员,你应该记住,这是一项需要付出巨大努力的相当大的事业。

当然,涉及的开发商可能比我们想象的更有能力。但莱纳斯·托瓦尔兹肯定不会犯错。😊

如果你打算参与其中的任何一项,并在未来提交拉取请求,我建议你先做一些研究,知道你应该/不应该做什么。

途径:福罗尼克斯

类似文章