LinusTorvals发火:LinuxPR请求如果没有注释,不要提交过
2023-02-23 13:35 来源:IT之家 阅读量:8415 小 中
,Linux 6.3 合并窗口期已经于近日启动,Linus Torvalds 收到了大量的 pull request请求,整体而言本次窗口期的各项工作开展顺利。
Linus Torvalds 也针对本次合并窗口期出现的问题发表了他的看法,IT之家翻译如下:
我收到了这样一条 PR 请求:“将‘for-linus / hardening’分支合并到‘for-next / hardening’”。
整个请求信息只有这一行短短的文字。
我之前已经多次强调,现在显然需要再次强调:如果你懒得解释为什么需要合并,那么这条合并请求就应该抛弃。
这确实应该是每个开发人员都应该牢记的规则,而不只是个空头文字。
我再次强调一遍:如果你无法为合并请求提供合理的解释或者说明,那么请不要提交。
真的就这么简单。在不解释为什么存在这些合并的情况下,绝对 * 永远 * 没有合并的借口。
在这种情况下,我真的认为如果合并请求都没有注释,那么合并还有什么意义!
如果你觉得有理由的话,请说出来!让合并提交看起来更加合理。
因为现在它看起来完全没有意义。我真的 * 厌恶 * 毫无意义的合并。它们只会让历史看起来更糟,更难读懂。
郑重声明:此文内容为本网站转载企业宣传资讯,目的在于传播更多信息,与本站立场无关。仅供读者参考,并请自行核实相关内容。
相关文章
热门
聚焦
美股异动CoinbaseCOIN.US涨超6
智通财经APP获悉,周三,Coinbase在公布业绩后股价走...更多>
人民币兑美元中间价报6.9028元,下调26
今日人民币兑美元中间价6.9028元,较上一交易日的6.87...更多>
珠江股份回复上交所:置出资产组计提大额跌价准
:2月27日,广州珠江发展集团股份有限公司回复上交所关于重大...更多>
直面青年租房的“难事儿”
党的二十大报告强调,加快建立多主体供给、多渠道保障、租购并举...更多>
报告称谷歌PlayStore上23.9%的儿
,Comparitech公布的最新研究表明,谷歌PlaySt...更多>
英伟达第四财季营收60.51亿美元超出分析师
北京时间2月23日早间消息,英伟达今日公布了该公司的2023...更多>
复旦MOSS距离ChatGPT还有多远?
最近上热搜的MOSS,不是《流浪地球2》中量子计算机550W...更多>
95后大专生!加入A股最年轻总经理阵营,独董
*ST文化的人事变动又来了,这次公司选出了一名1995年出生...更多>
深交所向唐源电气发出关注函要求进一步说明股东
中国财富通2月21日-深交所向唐源电气发出关注函,针对公司实...更多>
奥拓电子调研纪要:预计今年主要在XR虚拟拍摄
中国财富通2月21日-奥拓电子披露投资者关系活动记录表,公司...更多>