V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
• 请不要在回答技术问题时复制粘贴 AI 生成的内容
C5H12O5
V2EX  ›  程序员

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

  •  
  •   C5H12O5 ·
    C5H12O5 · 1 月 13 日 · 3728 次点击

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

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

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

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

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

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

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

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

    不沟通不改超过一定时间 close
    aojunhao123
        24
    aojunhao123  
       1 月 14 日
    开源合作更多是人与人之间的沟通,而非代码。一个 pr 如果不能很好的说明为何做出改动的上下文,那其实维护者也没必要去跟进
    关于   ·   帮助文档   ·   自助推广系统   ·   博客   ·   API   ·   FAQ   ·   Solana   ·   1167 人在线   最高记录 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 23ms · UTC 23:39 · PVG 07:39 · LAX 15:39 · JFK 18:39
    ♥ Do have faith in what you're doing.