V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
nevill
V2EX  ›  程序员

分享一下你的 PR 在合并过程中发生的故事吧

  •  
  •   nevill · 2019-10-01 01:50:28 +08:00 · 3351 次点击
    这是一个创建于 1882 天前的主题,其中的信息可能已经有所发展或是发生改变。
    先说我自己。

    某个 PR 在半年前提出,review 过后一直没动静,半年后另外一哥们冒出来说要不要他再来一版,然后某大佬注意到了,赶紧 rebase,review 过后就合并了。

    嗯,半年。
    8 条回复    2019-10-02 17:57:50 +08:00
    lspvic
        1
    lspvic  
       2019-10-01 03:24:02 +08:00 via Android   ❤️ 1
    一次用 python 画图,发现 matplotlib 里文档描述有问题,就提了个 pr,然后大佬说我改的文字里丢了几个空格,他补了个 commit 到我自己 fork 库的分支上,还说 I hope you do not mind。当时真的有些震惊,我就是改了一点点,他还帮改我的 pr,提交 commit 到我的 fork 库,多麻烦,有这功夫他自己就可以直接改掉了,我肯定也觉得正常的。
    大佬们对别人的贡献那怕是一点点微小😁的工作也是极其尊重的。从此,merge,rebase 别人代码非常小心,尤其是冲突特别多的时候,生怕把别人的 commit 丢掉或者 squash 掉。
    orzorzorzorz
        2
    orzorzorzorz  
       2019-10-01 07:15:39 +08:00   ❤️ 1
    https://github.com/ant-design/ant-design/issues/12743#issuecomment-432063358
    这是我头一个 pr,当时是什么都不懂,只知道怎么加这个属性。后续可以点进相关 pr 看看,各路大佬都在帮我,指教 rebase 的、指教更新 snapshots 的...现在看来当时的我还是太浮躁,好在极幸运地有人带路,才能克服 pr 恐惧感。这点我到现在还记着,有萌新来 pr,都会耐着性子教他,希望他也能想我一般幸运
    sunjourney
        3
    sunjourney  
       2019-10-01 08:15:00 +08:00
    先本地 rebase 完重新 pr
    xiangyuecn
        4
    xiangyuecn  
       2019-10-01 08:18:40 +08:00
    别人说问题要用英文来写,那就用英文写喽,谷歌+百度混合翻译😂
    别人说要写测试,那就写测试喽😂
    别人说代码不规范,同一个文件里面的那些旧代码是老规范,新写的要改成新规范写😂
    别人说这个地方看不懂,重写😂
    raysonx
        5
    raysonx  
       2019-10-01 12:08:37 +08:00 via Android
    经历过维护者关闭我的 PR,并在我的 PR 基础上修改以自己名义提交这种事。
    claysec
        6
    claysec  
       2019-10-01 22:23:05 +08:00
    @orzorzorzorz 我也是。第一次提 pr 的时候什么都不懂。他也很耐心的教我。
    ericgui
        7
    ericgui  
       2019-10-02 04:11:50 +08:00
    我的故事不是 PR,issue

    某个流行的 PHP wechat 库,用了一个更高版本的依赖,导致安装失败。

    我提了一个 issue,然后此人第一时间关闭 issue,然后立刻升级了一个小版本,修复此 bug。也就是说,此事就此掩盖过去了。当时是一个周五的晚上。


    所以我能说什么呢,虚伪也不过如此吧。

    拜托,出 bug 不是很正常吗?不丢人,但仍然故意掩盖此事,不知道说什么好了。
    such
        8
    such  
       2019-10-02 17:57:50 +08:00 via iPhone
    @ericgui 老哥,是哪个 wechat 库啊
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   实用小工具   ·   882 人在线   最高记录 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 24ms · UTC 21:41 · PVG 05:41 · LAX 13:41 · JFK 16:41
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.