• 请不要在回答技术问题时复制粘贴 AI 生成的内容
C5H12O5
V2EX  ›  程序员

不是,开源项目碰到零交流连提多个 PR 的要怎么办

  •  
  •   C5H12O5 ·
    PRO
    ·
    C5H12O5 · Jan 13 · 5001 views
    This topic created in 301 days ago, the information mentioned may be changed or developed.

    这段时间摸了个开源项目,基本都是在自娱自乐,也就在 V 站等几个论坛发过帖宣传了下。

    今天突然有同学连提了好几个 PR ,刚看见我还挺高兴的,但点进去看了发现代码质量真的不高,某些地方甚至像是 AI 生成的?

    这种情况大佬们碰到过吗,都是怎么处理的,我感觉放着不管的话他还会继续提更多的 PR🤦

    24 replies  •  2026-01-14 18:31:06 +08:00
    ZeroClover
    ZeroClover
    Jan 13❤️1
    1
    有时间有心情就指出什么地方需要修改
    没时间就 Close
    gullitintanni
    2
    open source 不等于 open collaboration ,该怎么处理全看你自己心情。如果你想跟他合作,那就花点时间沟通。否则无视就行。
    seeu2ex
    seeu2ex
    Jan 13
    3
    怎么看出来 ai 写得
    C5H12O5
    C5H12O5
    OP
    PRO
    Jan 13
    4
    @gullitintanni 好的,决定统一回复一篇后,把后续的都 Close 掉了,刚发现他甚至没点个 Star 😂
    kdwnil
    kdwnil
    Jan 13❤️1
    5
    质量不高的直接 close ,给双方都节省时间
    JoeJoeJoe
    JoeJoeJoe
    Jan 13
    6
    可能是刷数据?
    mooyo
    mooyo
    Jan 13
    7
    我们的项目之前也碰到了,几十个 PR 一个人通过 AI 提的,后面给他全 close 了。
    Astrian
    Astrian
    Jan 13 via Android
    8
    感觉就是有人为了刷存在感+开源贡献度就整个 AI 到处拉屎
    midsolo
    midsolo
    Jan 13
    9
    有专门刷开源项目 pr 的工作室,一句话都不说,代码全部由 AI 生成
    C5H12O5
    C5H12O5
    OP
    PRO
    Jan 13
    10
    @seeu2ex 有很多莫名的逻辑,比如为了添加一个缓存功能,引入了 sqlite ,明明是首次引入,却还加了段 // Migrate legacy schema 的逻辑,去迁移一个不存在的东西
    gongym
    gongym
    Jan 13
    11
    看了下提交的代码,不太好确认为恶意,不过确实很像使用 cursor 编写的

    我认为提交的同学应该就是在实际使用过程中发现了一些需要优化的地方,出于好心提交的 PR
    livib
    livib
    Jan 13
    12
    RealYourDad
    13
    质量太低了,回复一下,关了或者就晾在那里就行
    JoeDH
    JoeDH
    Jan 14
    14
    你的项目你做主啊,不要什么垃圾代码都收啊
    diudiuu
    diudiuu
    Jan 14
    15
    哈哈哈哈,我觉的又是哪个培训机构或者视频博主给教的,怎么参与开源项目,让自己简历更丰富,有人要找工作用到你了,你快给说明上加上那几个 pr 人名字,这几个人是混的,千万别要
    iorilu
    iorilu
    Jan 14
    16
    这年头 AI 代码正常把

    主要看代码有没有用阿
    C5H12O5
    C5H12O5
    OP
    PRO
    Jan 14
    17
    @iorilu 哈哈哈,不排斥 AI ,我自己也用的飞起。主要就是没用,或者说质量低,像是新手 vide coding 出来的东西改也不改就放上来了
    fstar
    fstar
    Jan 14
    18
    和他说他的代码有什么问题,让他改。

    不过通常都是从设计思路就不对了,一般他改了之后也是质量差,合入不了的。

    所以可以给晾个几天,然后给它关闭咯。
    fstar
    fstar
    Jan 14
    19
    正常是贡献者要先创建个 issue 的,说明自己想加个什么东西,维护者觉得和自己项目的方向一致,可以考虑让贡献者去实现。然后他提 MR ,然后一起看看讨论。
    最怕那种上来就提个 MR ,是我不想要的功能的。。。
    HojiOShi
    HojiOShi
    Jan 14
    20
    这种都还好,最恶心的是一些纯 spam 的账号,给你发的 PR 就是乱改点文档,issues 也是乱写几个字。
    Nexora
    Nexora
    Jan 14
    21
    你可以在 PR 里面沟通,让他怎么改动,你可以可以改动 PR 然后合并的。
    luyipao
    luyipao
    Jan 14
    22
    抵制 AIcode 吗,没必要。质量低就让他改,不改就 close 呗
    realpg
    realpg
    Jan 14
    23
    直接写意见 低质量代码 AI 生成包含无意义代码 让他整改就完了

    不沟通不改超过一定时间 close
    aojunhao123
    24
    开源合作更多是人与人之间的沟通,而非代码。一个 pr 如果不能很好的说明为何做出改动的上下文,那其实维护者也没必要去跟进
    About   ·   Help   ·   Advertise   ·   Blog   ·   API   ·   FAQ   ·   Privacy   ·   Solana   ·   2336 Online   Highest 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 123ms · UTC 13:30 · PVG 21:30 · LAX 05:30 · JFK 08:30
    ♥ Do have faith in what you're doing.