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’”。

整个请求信息只有这一行短短的文字。

我之前已经多次强调,现在显然需要再次强调:如果你懒得解释为什么需要合并,那么这条合并请求就应该抛弃。

这确实应该是每个开发人员都应该牢记的规则,而不只是个空头文字。

我再次强调一遍:如果你无法为合并请求提供合理的解释或者说明,那么请不要提交。

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

在这种情况下,我真的认为如果合并请求都没有注释,那么合并还有什么意义!

如果你觉得有理由的话,请说出来!让合并提交看起来更加合理。

因为现在它看起来完全没有意义。我真的 * 厌恶 * 毫无意义的合并。它们只会让历史看起来更糟,更难读懂。

郑重声明:此文内容为本网站转载企业宣传资讯,目的在于传播更多信息,与本站立场无关。仅供读者参考,并请自行核实相关内容。

热门

一键全自动换电——泰州市首家蔚来第三代换电站在兴化戴南上线
一键全自动换电——泰州市首家蔚来第三代换电站

一辆新能源汽车稳稳地停到蔚来充换电一体站的停车平台上,汽车下部的换电装置自动开启,卸下用完电的电池,换上充足电的...更多>

azg集团多维度赋能品牌,助力打造可持续产业生态
azg集团多维度赋能品牌,助力打造可持续产业

日前,由azg集团与Bubs贝儿联合举办的“Bubs贝儿纯净营养,守护新队友”五一特别场线下亲子活动圆满结束,门...更多>

喜报!中诚信托荣获深交所优秀债券投资交易机构奖项
喜报!中诚信托荣获深交所优秀债券投资交易机构

2月27日,深圳证券交易所组织召开2022年度债券监管发展业务座谈会,并表彰在服务国家重点战略、推进债券市场高质...更多>

银保监会就保险保障基金管理办法公开征求意见
银保监会就保险保障基金管理办法公开征求意见

为促进保险业稳健经营和高质量发展,更好发挥保险保障基金的积极作用,维护保单持有人合法权益,银保监会商有关部门对2...更多>

人民银行:指导银行将个人信贷延期还款政策落到实处
人民银行:指导银行将个人信贷延期还款政策落到

2022年第一季度金融统计数据新闻发布会现场。人民网黄盛摄人民网北京4月14日电(黄盛)今日,中国人民银行(以下...更多>

聚焦

发放消费券、举办消费节、推动线上线下消费融合,近期一批促消费措施密集推出—— 为消费升温积聚合力
美股异动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日-奥拓电子披露投资者关系活动记录表,公司...更多>