
这段时间摸了个开源项目,基本都是在自娱自乐,也就在 V 站等几个论坛发过帖宣传了下。
今天突然有同学连提了好几个 PR ,刚看见我还挺高兴的,但点进去看了发现代码质量真的不高,某些地方甚至像是 AI 生成的?
这种情况大佬们碰到过吗,都是怎么处理的,我感觉放着不管的话他还会继续提更多的 PR
1 ZeroClover 1 天前 有时间有心情就指出什么地方需要修改 没时间就 Close |
2 gullitintanni 1 天前 open source 不等于 open collaboration ,该怎么处理全看你自己心情。如果你想跟他合作,那就花点时间沟通。否则无视就行。 |
3 seeu2ex 1 天前 怎么看出来 ai 写得 |
4 C5H12O5 OP @gullitintanni 好的,决定统一回复一篇后,把后续的都 Close 掉了,刚发现他甚至没点个 Star |
5 kdwnil 1 天前 质量不高的直接 close ,给双方都节省时间 |
6 JoeJoeJoe PRO 可能是刷数据? |
7 mooyo 1 天前 我们的项目之前也碰到了,几十个 PR 一个人通过 AI 提的,后面给他全 close 了。 |
8 Astrian 1 天前 via Android 感觉就是有人为了刷存在感+开源贡献度就整个 AI 到处拉屎 |
9 midsolo 1 天前 有专门刷开源项目 pr 的工作室,一句话都不说,代码全部由 AI 生成 |
10 C5H12O5 OP @seeu2ex 有很多莫名的逻辑,比如为了添加一个缓存功能,引入了 sqlite ,明明是首次引入,却还加了段 // Migrate legacy schema 的逻辑,去迁移一个不存在的东西 |
11 gongym 1 天前 看了下提交的代码,不太好确认为恶意,不过确实很像使用 cursor 编写的 我认为提交的同学应该就是在实际使用过程中发现了一些需要优化的地方,出于好心提交的 PR |
12 livib 1 天前 Claude  |
13 RealYourDad 1 天前 质量太低了,回复一下,关了或者就晾在那里就行 |
14 JoeDH 1 天前 你的项目你做主啊,不要什么垃圾代码都收啊 |
15 canteon 23 小时 55 分钟前 哈哈哈哈,我觉的又是哪个培训机构或者视频博主给教的,怎么参与开源项目,让自己简历更丰富,有人要找工作用到你了,你快给说明上加上那几个 pr 人名字,这几个人是混的,千万别要 |
16 iorilu 23 小时 47 分钟前 这年头 AI 代码正常把 主要看代码有没有用阿 |
17 C5H12O5 OP @iorilu 哈哈哈,不排斥 AI ,我自己也用的飞起。主要就是没用,或者说质量低,像是新手 vide coding 出来的东西改也不改就放上来了 |
18 fstar 23 小时 6 分钟前 和他说他的代码有什么问题,让他改。 不过通常都是从设计思路就不对了,一般他改了之后也是质量差,合入不了的。 所以可以给晾个几天,然后给它关闭咯。 |
19 fstar 23 小时 3 分钟前 正常是贡献者要先创建个 issue 的,说明自己想加个什么东西,维护者觉得和自己项目的方向一致,可以考虑让贡献者去实现。然后他提 MR ,然后一起看看讨论。 最怕那种上来就提个 MR ,是我不想要的功能的。。。 |
20 HojiOShi 22 小时 19 分钟前 这种都还好,最恶心的是一些纯 spam 的账号,给你发的 PR 就是乱改点文档,issues 也是乱写几个字。 |
21 hellojukay 21 小时 14 分钟前 你可以在 PR 里面沟通,让他怎么改动,你可以可以改动 PR 然后合并的。 |
22 luyipao 17 小时 17 分钟前 抵制 AIcode 吗,没必要。质量低就让他改,不改就 close 呗 |
23 realpg PRO 直接写意见 低质量代码 AI 生成包含无意义代码 让他整改就完了 不沟通不改超过一定时间 close |
24 aojunhao123 14 小时 22 分钟前 开源合作更多是人与人之间的沟通,而非代码。一个 pr 如果不能很好的说明为何做出改动的上下文,那其实维护者也没必要去跟进 |