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