V2EX = way to explore
V2EX 是一个关于分享和探索的地方
Sign Up Now
For Existing Member  Sign In
• 请不要在回答技术问题时复制粘贴 AI 生成的内容
C5H12O5
V2EX  ›  程序员

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

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

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

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

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

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

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