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

出品 | OSC开源社区(ID:oschina2013)

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

107967c60f6afc381fa8ec6735394833.png

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

6e8a169f1efe8f9cd31aa4b7d3facf7d.png

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

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

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

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

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

 
 
 
 
资料链接
 
 
清华学姐自学的Linux笔记,天花板级别!新版鸟哥Linux私房菜资料阿里大佬总结的《图解Java》火了,完整版PDF开放下载!
Alibaba官方上线!SpringBoot+SpringCloud全彩指南国内最强的SpringBoot+Vue全栈项目天花板,不接受反驳!

欢迎添加个人微信 itcodexy  进粉丝群或围观朋友圈

adb99dce1d5c6e45fb93827ec54e9bc5.gif

评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值