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 掉。 |
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,都会耐着性子教他,希望他也能想我一般幸运 |
3
sunjourney 2019-10-01 08:15:00 +08:00
先本地 rebase 完重新 pr
|
4
xiangyuecn 2019-10-01 08:18:40 +08:00
别人说问题要用英文来写,那就用英文写喽,谷歌+百度混合翻译😂
别人说要写测试,那就写测试喽😂 别人说代码不规范,同一个文件里面的那些旧代码是老规范,新写的要改成新规范写😂 别人说这个地方看不懂,重写😂 |
5
raysonx 2019-10-01 12:08:37 +08:00 via Android
经历过维护者关闭我的 PR,并在我的 PR 基础上修改以自己名义提交这种事。
|
6
claysec 2019-10-01 22:23:05 +08:00
@orzorzorzorz 我也是。第一次提 pr 的时候什么都不懂。他也很耐心的教我。
|
7
ericgui 2019-10-02 04:11:50 +08:00
我的故事不是 PR,issue
某个流行的 PHP wechat 库,用了一个更高版本的依赖,导致安装失败。 我提了一个 issue,然后此人第一时间关闭 issue,然后立刻升级了一个小版本,修复此 bug。也就是说,此事就此掩盖过去了。当时是一个周五的晚上。 所以我能说什么呢,虚伪也不过如此吧。 拜托,出 bug 不是很正常吗?不丢人,但仍然故意掩盖此事,不知道说什么好了。 |