V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
V2EX 提问指南
madNeal
V2EX  ›  问与答

如何看待 github 某些开源库的作者 pr 迟迟不处理

  •  
  •   madNeal · 2018-03-05 13:55:08 +08:00 · 3335 次点击
    这是一个创建于 2455 天前的主题,其中的信息可能已经有所发展或是发生改变。

    这里没有任何喷开源作者的意图,就是有的时候提了一些 pr,可是作者迟迟不处理,虽然别人可能是因为很忙,但我觉得花个几分钟看一下,如果觉得不行就直接拒掉,一直在那放着,看着好难受。

    第 1 条附言  ·  2018-03-05 16:57:50 +08:00
    不会是看到我发的帖子了吧 下午处理了。。。。。。
    第 2 条附言  ·  2018-03-05 16:58:35 +08:00
    不过还有个是另外一个人的
    19 条回复    2018-03-06 13:49:25 +08:00
    TaoSama
        1
    TaoSama  
       2018-03-05 14:07:56 +08:00 via Android
    这种不是应该去知乎问嘛 = =
    crysislinux
        2
    crysislinux  
       2018-03-05 14:13:33 +08:00
    几分钟哪儿够哦,除非是很小的项目和很小的 pr。
    madNeal
        3
    madNeal  
    OP
       2018-03-05 14:13:37 +08:00
    @TaoSama 感觉知乎很容易引战
    kslr
        4
    kslr  
       2018-03-05 14:13:55 +08:00
    一般先找找联系方式,发一封邮件; 如果几天还没回复,就 fork 一份重新发布新包。
    madNeal
        5
    madNeal  
    OP
       2018-03-05 14:15:11 +08:00
    @crysislinux 这里只是打个比方 其实我说的就是这种几分钟就能知道到底是 merge 还是 deny
    madNeal
        6
    madNeal  
    OP
       2018-03-05 14:16:25 +08:00
    @kslr 这里其实就是讨论一哈 其实他 merge 不 merge 对我没啥影响,就是看到 pr 一直不处理,总是要看一看
    aheadlead
        7
    aheadlead  
       2018-03-05 14:26:30 +08:00
    你提 pr 前和作者沟通了吗?
    wsy2220
        8
    wsy2220  
       2018-03-05 14:27:49 +08:00
    有过提 pr 两年后才被 merge 的经历...
    madNeal
        9
    madNeal  
    OP
       2018-03-05 14:31:10 +08:00
    @aheadlead 难道不是直接在 issue 里面讨论么,而且有的 pr 很小,就是那种很明显的错误
    oott123
        10
    oott123  
       2018-03-05 14:35:32 +08:00   ❤️ 2
    接受你的 PR,意味着要为你这段代码负责,这个事情是需要考虑的。
    比如你的代码尚可,但我此时又不想维护,close 显然不够礼貌,而 merge 要花的精力我接受不起,那就只能放一边……
    chinvo
        11
    chinvo  
       2018-03-05 14:37:02 +08:00
    除了 @oott123 #10 还有 CLA 的问题、WIP 的问题
    akira
        12
    akira  
       2018-03-05 14:45:20 +08:00
    想起很多年前给 一个项目 提过 issue 和修改代码,半天不到作者就集成然后发布新版本了,太勤奋了
    aheadlead
        13
    aheadlead  
       2018-03-05 15:01:42 +08:00
    @madNeal 我以为你这是几百上千行的大 commit
    airyland
        14
    airyland  
       2018-03-05 15:11:44 +08:00
    维护开源是件很费精力的事,对我来说,很简单的拼写错误什么的会直接合并,其他的看有没有时间审查代码,也可能一拖就半年。
    wekw
        15
    wekw  
       2018-03-05 15:37:13 +08:00
    多个项目的维护者来现身说法:顾不上每天检查,非 bug 的 PR 一般懒得处理,我几个月可能会冲动一次处理一波。
    xwhxbg
        16
    xwhxbg  
       2018-03-05 17:27:51 +08:00
    这让我想起之前我给 upterm 提 issue,挂了半年没人理,前一阵子看,结果发现一个新 issue,里面写着

    这个 repo 的主要维护者发生了 tragic death。。。。。
    vardarling
        17
    vardarling  
       2018-03-05 19:48:38 +08:00 via iPhone
    几千行的改动涉及 10+文件 看了一眼就懒得管了🌚
    feverzsj
        18
    feverzsj  
       2018-03-05 19:51:21 +08:00
    知道 linus 为什么老要骂人吗?都是被无下限的 pr 逼的
    itgoyo
        19
    itgoyo  
       2018-03-06 13:49:25 +08:00
    你可以尝试先邮件联系作者,而且难不难受看个人吧,毕竟不是所有都会在意那个的。还有很多作者都不怎么看自己项目的。
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   实用小工具   ·   1009 人在线   最高记录 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 21ms · UTC 19:40 · PVG 03:40 · LAX 11:40 · JFK 14:40
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.