这是一个创建于 3248 天前的主题，其中的信息可能已经有所发展或是发生改变。
 |
1
lincanbin 2015-12-31 21:32:54 +08:00
 这个人给我某个 repo 发了十几个 PR ，然后一堆鸡毛蒜皮的事（每个 Commit 一般就 10 行左右），有的还有语法错误，我看得出他想混 contributor 。 于是我花了几十分钟审阅了那十几个 PR ， Merge 了几个鸡毛蒜皮的、语法错误不是特别多的。 Merge 后又花了十几分钟擦干净他代码里一一些错误（这么几行代码都有一堆错误呢）  Merge 完了，这个人跑来 PR 下面这么回复我了：  |
 |
3
subpo 2015-12-31 21:36:56 +08:00
哈哈哈哈哈哈哈
|
 |
4
Troevil 2015-12-31 21:43:17 +08:00
|
 |
6
pupboss 2015-12-31 21:54:13 +08:00
@lincanbin 23333 他怎么在 PR 的代码里还加一行注释，说 Code by Tecflare http://www.tecflare.com
|
 |
7
Slienc7 2015-12-31 21:54:23 +08:00
|
|
8
lincanbin 2015-12-31 22:03:51 +08:00
@pupboss 你还跑去视奸了……
他几乎所有代码都要加这个， PR 里明确说了这个注释不让删除。 有个 PR 直接改 ReadMe ，写了自己的名字和网址和自己的简介…… 我自己都没给所有代码加我自己的名字呢…… 反正是鸡毛蒜皮的 PR 我也就不合并了…… |
 |
9
ilotuo 2015-12-31 22:07:54 +08:00
哈哈哈哈哈学习了
我的 190 多个 stars 你们颤抖了吗! |
 |
10
raincious 2015-12-31 22:37:44 +08:00
当即决定先下手围墙
 |
 |
11
est 2015-12-31 23:20:25 +08:00
脑洞大开！
|
 |
12
mzer0 2015-12-31 23:37:30 +08:00
你们集体在 github 上这样做的话, 老外对中国人的印象会变差的. 下次中国人发 PR, 人家就不收了......
|
 |
14
bdbai 2016-01-01 00:01:24 +08:00
完了完了， pr 刚发出去才发现重了...
http://ww3.sinaimg.com/large/a04a339djw1ezja7l0atqj20zk0k0n0g.jpg |
|
15
lincanbin 2016-01-01 01:01:47 +08:00
@zsx 是我自己的疏忽， merge pull request 的时候因为那个人 commit 的代码没有合理缩进，所以我没看出那个低级的语法错误。我还以为那个人 commit 前肯定确保了自己的代码是能跑的呢……
实际上那个人的不少 PR 都是从一些地方复制过来的…… 我觉得没事就别去给人提无关痛痒的 PR 了， merge 后要走的流程好多的（虽然我一般都懒得走）。 |
 |
16
zts1993 2016-01-01 13:27:14 +08:00 via Android
简直醉了
|
Select Language