Linus再发飙:这就是一堆垃圾!

Linux 6.3 内核的合并窗口已开启,Linus Torvalds 也收到了大量的 PR,目前总体看来正在有序进行。但 Linus 对部分合并请求的日志信息非常不满:“我之前就已经说过,很显然现在我需要再重复一次,如果你懒得解释为什么会存在这个合并请求,那么根据定义,这就是一堆垃圾!”

Linus再发飙:这就是一堆垃圾!

▲缺少注释的合并请求让 Linus Torvalds 很生气

Linus再发飙:这就是一堆垃圾!

Linus 认为,如果你不能解释清楚一个合并请求,那么就不要提交,这是很简单的道理。如果不解释提交合并请求的原因,那就是在生产垃圾。在这种情况下,Linus 觉得这种合并请求根本就不应该存在。

Linus 强调道,这是每个开发者都应该牢记的规则。如果有注释的话,那就在日志信息中写出来,让那个合并请求的提交看起来合理些。

现在这些没有注释的合并请求看起来完全没有意义,Linus 表示自己十分厌恶无意义的合并。它们只会让代码历史看起来更糟糕,更难读。

最后的结果是,这次合并是对之前的 Linux 6.2 的修复,相关的开发者会努力在未来更清楚地表达合并请求的注释。

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


传送门

原文始发于微信公众号(开源技术专栏):Linus再发飙:这就是一堆垃圾!

版权声明:本文内容由互联网用户自发贡献,该文观点仅代表作者本人。本站仅提供信息存储空间服务,不拥有所有权,不承担相关法律责任。如发现本站有涉嫌侵权/违法违规的内容, 请发送邮件至 举报,一经查实,本站将立刻删除。

文章由极客之音整理,本文链接:https://www.bmabk.com/index.php/post/120628.html

(0)
小半的头像小半

相关推荐

发表回复

登录后才能评论
极客之音——专业性很强的中文编程技术网站,欢迎收藏到浏览器,订阅我们!