分享一下你的 PR 在合并过程中发生的故事吧 - V2EX
V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
请不要在回答技术问题时复制粘贴 AI 生成的内容
nevill

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

  •  
  •   nevill 2019 年 10 月 1 日 4023 次点击
    这是一个创建于 2399 天前的主题,其中的信息可能已经有所发展或是发生改变。
    先说我自己。

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

    嗯,半年。
    8 条回复    2019-10-02 17:57:50 +08:00
    lspvic
        1
    lspvic  
       2019 年 10 月 1 日 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 月 1 日   1
    https://github.com/ant-design/ant-design/issues/12743#issuecomment-432063358
    这是我头一个 pr,当时是什么都不懂,只知道怎么加这个属性。后续可以点进相关 pr 看看,各路大佬都在帮我,指教 rebase 的、指教更新 snapshots 的...现在看来当时的我还是太浮躁,好在极幸运地有人带路,才能克服 pr 恐惧感。这点我到现在还记着,有萌新来 pr,都会耐着性子教他,希望他也能想我一般幸运
    sunjourney
        3
    sunjourney  
       2019 年 10 月 1 日
    先本地 rebase 完重新 pr
    xiangyuecn
        4
    xiangyuecn  
       2019 年 10 月 1 日
    别人说问题要用英文来写,那就用英文写喽,谷歌+百度混合翻译
    别人说要写测试,那就写测试喽
    别人说代码不规范,同一个文件里面的那些旧代码是老规范,新写的要改成新规范写
    别人说这个地方看不懂,重写
    raysonx
        5
    raysonx  
       2019 年 10 月 1 日 via Android
    经历过维护者关闭我的 PR,并在我的 PR 基础上修改以自己名义提交这种事。
    claysec
        6
    claysec  
       2019 年 10 月 1 日
    @orzorzorzorz 我也是。第一次提 pr 的时候什么都不懂。他也很耐心的教我。
    ericgui
        7
    ericgui  
       2019 年 10 月 2 日
    我的故事不是 PR,issue

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

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


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

    拜托,出 bug 不是很正常吗?不丢人,但仍然故意掩盖此事,不知道说什么好了。
    such
        8
    such  
       2019 年 10 月 2 日 via iPhone
    @ericgui 老哥,是哪个 wechat 库啊
    关于     帮助文档     自助推广系统     博客     API     FAQ     Solana   &nbs; 2746 人在线   最高记录 6679       Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 40ms UTC 10:03 PVG 18:03 LAX 03:03 JFK 06:03
    Do have faith in what you're doing.
    ubao msn snddm index pchome yahoo rakuten mypaper meadowduck bidyahoo youbao zxmzxm asda bnvcg cvbfg dfscv mmhjk xxddc yybgb zznbn ccubao uaitu acv GXCV ET GDG YH FG BCVB FJFH CBRE CBC GDG ET54 WRWR RWER WREW WRWER RWER SDG EW SF DSFSF fbbs ubao fhd dfg ewr dg df ewwr ewwr et ruyut utut dfg fgd gdfgt etg dfgt dfgd ert4 gd fgg wr 235 wer3 we vsdf sdf gdf ert xcv sdf rwer hfd dfg cvb rwf afb dfh jgh bmn lgh rty gfds cxv xcv xcs vdas fdf fgd cv sdf tert sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf shasha9178 shasha9178 shasha9178 shasha9178 shasha9178 liflif2 liflif2 liflif2 liflif2 liflif2 liblib3 liblib3 liblib3 liblib3 liblib3 zhazha444 zhazha444 zhazha444 zhazha444 zhazha444 dende5 dende denden denden2 denden21 fenfen9 fenf619 fen619 fenfe9 fe619 sdf sdf sdf sdf sdf zhazh90 zhazh0 zhaa50 zha90 zh590 zho zhoz zhozh zhozho zhozho2 lislis lls95 lili95 lils5 liss9 sdf0ty987 sdft876 sdft9876 sdf09876 sd0t9876 sdf0ty98 sdf0976 sdf0ty986 sdf0ty96 sdf0t76 sdf0876 df0ty98 sf0t876 sd0ty76 sdy76 sdf76 sdf0t76 sdf0ty9 sdf0ty98 sdf0ty987 sdf0ty98 sdf6676 sdf876 sd876 sd876 sdf6 sdf6 sdf9876 sdf0t sdf06 sdf0ty9776 sdf0ty9776 sdf0ty76 sdf8876 sdf0t sd6 sdf06 s688876 sd688 sdf86