别再给我发超大PR了;一篇吐槽
摘要
这篇吐槽批评了软件开发中流行的大型AI生成的拉取请求趋势,提倡使用更小、易于审查的改动来提升代码质量和审查者的体验。
暂无内容
查看缓存全文
缓存时间: 2026/08/15 00:30
# 别再给我发巨型PR了;一场吐槽
来源:https://getsmall.xyz/post/cmstjfl9l000if70ljmpzr4va
我受够了,伙计。我厌倦了审查那些一、两、三千行代码的PR,就因为某个智能体"一次性解决了整个问题"。要求小PR从来不是因为它更容易编写,而始终是为了审查者的便利。AI对行业来说是巨大的福音,但对于审查者和维护者却正在变成沉重负担。或许我只是个对着云彩嚷嚷的老古董,但拜托了,别再给我发巨型PR。
最近我多次听到这样的论点:"没有完整改动就无法生效"或者"如果不放入所有差异代码,程序什么也做不了",嗯?那又怎样?小PR的意义不在于获得小巧、独立、完成的作品,而是为了得到小块、易消化、可审查、易理解的工作成果。虽然缺乏数据支持,但我大胆推测:理解一段代码所需的时间随着代码行数增加呈指数级增长。只因你想打包完整功能上线,就耗费我数倍时间,这实在让人难以愉悦。
顺便说一句,我不需要50行的注释。当然,要为函数做文档,给我jsdoc、rustdoc、javadoc这些优秀工具产出的注释。但千万别为解释变量为何命名为`is_logged_in`就写五行注释。如果变量命名得当,十有八九我能理解它的作用。如果命名不当需要注释,不如把变量名改得更好些。
最后,对你们这些认为"直接用AI理解代码就行"的AI至上者(这里的"理解"指https://en.wikipedia.org/wiki/Grok,不是https://grok.com/那个产品),你们只是在浪费算力重新消化AI生成的代码。"但我用的是另一个模型做审查",好吧,那何必还要提交给人类审查?或许可以让你珍视的AI先把代码分块整理好,等我们这些凡人审查完再由它接手?
看——AI是伟大的工具,它确实能加速开发并提升代码质量,但当初React推出时,我们并没有因为"React编写更快、更易读"就接受更大的PR,现在为何要妥协?
*补遗:你故意发巨型PR,是不是想让我审到一半放弃直接通过?若是如此,干得漂亮。真·干得漂亮。*
相似文章
寻求帮助
一篇讽刺性的开源维护者抱怨,针对自以为是的用户、AI生成的拉取请求,以及被迫采纳流行但不健康的开发实践。
我不再需要你的 PR
一位开源维护者解释为何现在更偏爱由 LLM 生成的代码,而非社区 PR:AI 辅助能降低风险与摩擦,将贡献价值转向反馈、缺陷报告与设计讨论。
用这一个简单技巧让代码审查重新可行
文章提出使用堆叠分支(小型、顺序的拉取请求)来使审查AI生成的代码更易管理和高效,解决常见的大型、难以审查的差异问题。
@github:AI 编码代理可以快速生成功能,但可能将其打包成一个巨大的、难以审查的拉取请求。下次,…
GitHub 建议,AI 编码代理可以快速生成功能,但常常创建大型、难以审查的 PR,并建议将工作拆分为专注的、有序的 PR。
构建了一个AI PR审核工作流,可生成实际修复PR而不仅仅是评论
作者构建了一个AI驱动的PR审核工作流,可生成实际修复PR而不仅仅是评论,声称比CodeRabbit便宜6倍,且在处理大型PR时更准确。