This topic created in 3786 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