停止给我发巨大的 PR;一场愤怒
我累了,老板。我厌倦了审查一、二、三千行的 PR,因为某个代理人能够“一次性解决整个问题”。小 PR 从来不是因为它们更容易编写而被要求的,而是为了审查者的利益。人工智能对行业来说是如此的福音,但对于审查者和维护者来说,它正在变成一种负担。也许我只是个对着云喊叫的老头,但请,停止给我发送巨大的 PR。我最近多次听到这样的论点,“没有整个变更就无法工作”或“如果我们不全部提交,代码将不会起作用”,好吧?很好吗?小 PR 的要点并不一定是要有小的、单独的、完成的产品。它是为了拥有小的、易于消化的、可审查的、易理解的工作片段。我对此没有数据,所以我会胡乱推测,完全理解一段代码所需的时间与代码的行数成指数增长。因为你想要发布完整的功能而花费我越来越多的时间并没有带来快乐。在这方面,我也不需要 50 行的注释。当然,文档函数,给我 jsdoc、rustdoc、javadoc,所有这些都很好。绝对不要给我 5 行注释来解释为什么一个变量被命名为 is_logged_in。如果你的变量命名得好,10 次中有 9 次我能理解它的作用。如果你的变量命名不好而需要注释,那就把你的变量命名得更好。最后,对于那些说“只需用 AI 理解(而不是理解)它,兄弟”的 AI 追随者们,你们在重新摄取已经由 AI 制作的代码时浪费了你们的令牌。“我用不同的模型进行审查,”好吧,太好了,那你为什么要把它提交给人审查呢?也许,你可以让你宝贵的 AI 为我们这些凡人分开处理,然后在我们审查后再让它审查?看——人工智能是一个很好的工具,它确实帮助事情更快推进,并能改善代码,但当 React 出现时,我们并没有接受更大的 PR,因为“React 更容易编写和阅读”,那我们现在为什么要这样做?附录:你是故意做巨大的 PR,让我半途放弃并批准它吗?如果是这样,干得好。真是干得好。
本站免费、广告极少。如果觉得有帮助,可以请我们喝杯咖啡 —— 任何金额都对持续运营有实际帮助。
☕请我喝杯咖啡