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

2019-10-01 01:50:28 +08:00
 nevill
先说我自己。

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

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

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

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


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

拜托,出 bug 不是很正常吗?不丢人,但仍然故意掩盖此事,不知道说什么好了。
such
2019-10-02 17:57:50 +08:00
@ericgui 老哥,是哪个 wechat 库啊

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/605812

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX